raamdev / wp-redirects

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

Sorting by Hits or Last Access time excludes Redirects created with old version of plugin #10

Closed raamdev closed 10 years ago

raamdev commented 10 years ago

Forked from #8.

Need to finish #9 before working on this bug.


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. .