search

raamdev / wp-redirects

Redirects (a new Post Type) for WordPress®.
1 stars 2 forks source link

Keep track of redirect hits and last access time #8

Closed raamdev closed 10 years ago

raamdev commented 10 years ago

Resolves Issue #4.

Keeps track of redirect hits and last access time. Both columns are sortable and the hit counter can be reset.

2014-09-11_00-59-46

2014-09-11_01-01-40

This should be thoroughly tested to ensure backwards compatibility with existing Redirects created with an earlier version of WP-Redirects.

raamdev commented 10 years ago

I just tested these changes against Redirects that were added with a previous version of WP-Redirects and while there are no errors, there is a bad UX problem that should be fixed:

Redirects created with a previous version show up with blank Hits and Last Access values:

2014-09-11_01-08-34

That's fine, except when you then click on either column header to sort by that column. What happens is the sort algorithm doesn't have any data for which to sort those rows, so they get excluded from the query altogether and disappear from the list:

2014-09-11_01-09-08

A routine should be added to detect when upgrading from an older version and then insert default values for all of the existing redirects.

I'll work on that now.

raamdev commented 10 years ago

Merging with the above bug so that I can work on #9 before fixing the bug.