User tracking mod

[petro]

mc, ill take it to another thread. It looks like I have a tpl_top_html.php that looks just like the footer that my site is calling to not the footer.... wierd

[Woodymon]

A couple issues with the install sql

1. CREATE TABLE user_tracking... TYPE=MyISAM
should be ENGINE=MyISAM

That should fix mod related errors which show up in the logs
( includes/classes/db/mysql/query_factory.php on line 120).

2. And the sort value should be 999 (not 31) for
INSERT INTO configuration_group (`configuration_group_id`,.....

(and the uninstall sql posted to this thread adjusted accordingly).

3. Also the readme should say User Tracking Config now shows up in Tools menu (previously appeared in the Configuration menu)

Appreciate the updates to the mod to hide/show spider crawls.

[mc12345678]

A couple issues with the install sql

1. CREATE TABLE user_tracking... TYPE=MyISAM
should be ENGINE=MyISAM

That should fix mod related errors which show up in the logs
( includes/classes/db/mysql/query_factory.php on line 120).

2. And the sort value should be 999 (not 31) for
INSERT INTO configuration_group (`configuration_group_id`,.....

(and the uninstall sql posted to this thread adjusted accordingly).

3. Also the readme should say User Tracking Config now shows up in Tools menu (previously appeared in the Configuration menu)

Appreciate the updates to the mod to hide/show spider crawls.

- Item 1 is being corrected in an update.
- Item 2 an the update is going to use the next number instead of forcing to 999 (suggestion posted ages ago by Dr. Byte, as such I have tried to go through the entire readme to pull in any suggestions/hints/tips to incorporate into the next update, these will be added as well.)
- Item 3 noted and will update that in the documents. Last couple of updates were to specifically target some changes and did not include any form of comprehensive review.

This leads to the last non-bulleted item. During the comprehensive review it was identified that the code already partially hid spider crawls, the addition made is supposed to effectively hide them now, but the resulting "show spider crawls" does not truly show all of the spider crawls. This is an area that I would like to expand on and add some additional administrative controls to allow a store "reviewer" to be only allowed to do as permitted by the store owner. (Ie., may not be able to delete records, Hide from the reviewer the admin's activity or the activity of a particular individual (again would be logged, but not visible until the config switch was flipped allowing it to be seen), perhaps some other similar controls.) In review of the forum, there was guidance provided about what to enter into the footer to track various types of information, but have now seen that those hints were not incorporated, so would like to offer an admin switch to change the logging "preferences" as well.

I possibly could push out an interim update; however, have been wanting to not cycle the users of it, though I guess at least the SQL for a new install should be updated to help those newly coming on-board.

[mc12345678]

A couple issues with the install sql

2. And the sort value should be 999 (not 31) for
INSERT INTO configuration_group (`configuration_group_id`,.....

(and the uninstall sql posted to this thread adjusted accordingly).

Been thinking about this particular comment and trying to understand. I took a look through my cart's database and found that in some cases the sort_id followed the configuration_group_id; however, in some cases not. Where is there information associated with the way this should go as a "general programming" option? I've already modified the preliminary update to reflect the same sort_id as the configuration_group_id (whatever it may end up being based on the user's cart), but when I changed the setting in my own cart's database nothing changed in my admin panel.

Further related to the config section, I am not sure why it was changed to be in the area of the Tools menu and was also "confused" about why. It would seem to me that it should fall into the configuration menu and that the User Tracking Log potentially stay where it is. Anyone have design input while moving forwards with additional updates either from a usage side or to meet ZC standards? (I realize this could be a can of worms, but if making changes, should consider the larger audience than my own "little world".)

[mc12345678]

Just submitted an update to this plugin, may not be "available" until next weekend:

1. Added settings to Admin Config:
1.1) User visiting the User tracking page ability to delete log entries.
1.2) Ability to modify how much history to keep when purging/deleting entries using standard units of measure (hours, days, weeks, months (determined using 30 days)
1.3) Added option of data to submit into log based on user forum input.
1.4) Added a version field to the configuration utility.
2. Modified English Language File to show the duration and units of the record(s) being kept/purged.
3. Updated the shop's class.user_tracking observer to support logging of visits based on entries of the Zen-Cart user forum.
4. Changed the Update SQL file to reflect only those changes applicable with this upgrade (I.e. old update SQL which was only really applicable to upgrading older versions of User Tracking to an earlier version.
5. Added the number of unique visitors and spiders to the beginning of the list of sessions, so that it is now at top and bottom.
6. Hid all options to delete information if the admin has set the delete option to false, which means not only does it not display, but also that should not be able to use a get statement to delete the old data. The ability to delete all of the items overrides the inability to delete any one item. (ie., if delete all is on and delete IP set to off then delete IP will still be possible.)
7. Added uninstall_user_tracking.sql file.

Have not modified the logging/tracking to show more of the spiders that have been worked out of the log data. There are a few other improvements yet considered (such as an auto-installer) that have not been incorporated; however, there have been issues in the installation process that have been addressed in the forum but not incorporated into the files. A majority of that has been done.

[BlessIsaacola]

Thank you so much for the update. I tried to give this mod another try but failed with the SQL install. The following is the error message I received:

[14-Nov-2013 05:03:35 America/New_York] PHP Fatal error: 1064:You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '. '.php', '', 'tools', 'Y', 10000)' at line 1 :: /* Tables need to be ensured to include the prefix if it exists. */ INSERT INTO admin_pages (page_key, language_key, main_page, page_params, menu_key, display_on_menu, sort_order) VALUES ('UserTracking', 'BOX_TOOLS_USER_TRACKING', 'FILENAME_USER_TRACKING' . '.php', '', 'tools', 'Y', 10000); in /includes/classes/db/mysql/query_factory.php on line 120

Here are the two lines from the previous sql install:

INSERT INTO admin_pages (page_key, language_key, main_page, page_params, menu_key, display_on_menu, sort_order) VALUES ('UserTracking', 'BOX_TOOLS_USER_TRACKING', 'FILENAME_USER_TRACKING', '', 'tools', 'Y', 10000); 

INSERT INTO admin_pages (page_key, language_key, main_page, page_params, menu_key, display_on_menu, sort_order) VALUES ('UserTrackingConfig', 'BOX_TOOLS_USER_TRACKING_CONFIG', 'FILENAME_USER_TRACKING_CONFIG', '', 'tools', 'Y', 10001);

Here are the two lines from the current sql install:

INSERT INTO admin_pages (page_key, language_key, main_page, page_params, menu_key, display_on_menu, sort_order) VALUES ('UserTracking', 'BOX_TOOLS_USER_TRACKING', 'FILENAME_USER_TRACKING' . '.php', '', 'tools', 'Y', 10000); 

INSERT INTO admin_pages (page_key, language_key, main_page, page_params, menu_key, display_on_menu, sort_order) VALUES ('UserTrackingConfig', 'BOX_TOOLS_USER_TRACKING_CONFIG', 'FILENAME_USER_TRACKING_CONFIG' . '.php', '', 'tools', 'Y', 10001);

Not sure why the modification in the current install but definitely causing issue.

[mc12345678]

Thank you so much for the update. I tried to give this mod another try but failed with the SQL install. The following is the error message I received:

Here are the two lines from the previous sql install:

INSERT INTO admin_pages (page_key, language_key, main_page, page_params, menu_key, display_on_menu, sort_order) VALUES ('UserTracking', 'BOX_TOOLS_USER_TRACKING', 'FILENAME_USER_TRACKING', '', 'tools', 'Y', 10000); 

INSERT INTO admin_pages (page_key, language_key, main_page, page_params, menu_key, display_on_menu, sort_order) VALUES ('UserTrackingConfig', 'BOX_TOOLS_USER_TRACKING_CONFIG', 'FILENAME_USER_TRACKING_CONFIG', '', 'tools', 'Y', 10001);

Here are the two lines from the current sql install:

INSERT INTO admin_pages (page_key, language_key, main_page, page_params, menu_key, display_on_menu, sort_order) VALUES ('UserTracking', 'BOX_TOOLS_USER_TRACKING', 'FILENAME_USER_TRACKING' . '.php', '', 'tools', 'Y', 10000); 

INSERT INTO admin_pages (page_key, language_key, main_page, page_params, menu_key, display_on_menu, sort_order) VALUES ('UserTrackingConfig', 'BOX_TOOLS_USER_TRACKING_CONFIG', 'FILENAME_USER_TRACKING_CONFIG' . '.php', '', 'tools', 'Y', 10001);

Not sure why the modification in the current install but definitely causing issue.

Thank you for the detailed feedback. The change was made, because the assignment of the filename in the associated define file was not standard to ZC. (It previously included the extension.) That said, I did not properly format the SQL statement to concatenate the two strings. I'll have to provide an update.

In the meantime, and because I have not tested this statement, try replacing 'FILENAME_USER_TRACKING_CONFIG' . '.php' with:
CONCAT('FILENAME_USER_TRACKING_CONFIG', '.php')
And report back the result.
And the same thing with the one without CONFIG on the end.

[mc12345678]

Thank you for the detailed feedback. The change was made, because the assignment of the filename in the associated define file was not standard to ZC. (It previously included the extension.) That said, I did not properly format the SQL statement to concatenate the two strings. I'll have to provide an update.

In the meantime, and because I have not tested this statement, try replacing 'FILENAME_USER_TRACKING_CONFIG' . '.php' with:
CONCAT('FILENAME_USER_TRACKING_CONFIG', '.php')
And report back the result.
And the same thing with the one without CONFIG on the end.

Yeah, it really pays to test all aspects of a change...

So the SQL statement for adding the two menu options of tracking and user tracking config should not have been changed. I wrongly assumed that because I had changed the Define for the filename that I would have to append '.php' to the sql statement. So, yes an update will still be necessary to the install SQL to restore those two lines back to what they were:

INSERT INTO admin_pages (page_key, language_key, main_page, page_params, menu_key, display_on_menu, sort_order) VALUES ('UserTracking', 'BOX_TOOLS_USER_TRACKING', 'FILENAME_USER_TRACKING', '', 'tools', 'Y', 10000);

INSERT INTO admin_pages (page_key, language_key, main_page, page_params, menu_key, display_on_menu, sort_order) VALUES ('UserTrackingConfig', 'BOX_TOOLS_USER_TRACKING_CONFIG', 'FILENAME_USER_TRACKING_CONFIG', '', 'tools', 'Y', 10001);