Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Log Display Screen #120

Open
joshbetz opened this issue Oct 13, 2016 · 1 comment
Open

Fix Log Display Screen #120

joshbetz opened this issue Oct 13, 2016 · 1 comment

Comments

@joshbetz
Copy link
Contributor

The Syndication Logs screen is a little broken. It looks like we use the same query on every page and filter in PHP. This creates a couple problems

  • We only get the most recent logs. On an active site, the related logs can scroll out of range before you even get a chance to look.
  • You can't get all the, for example, errors on a single page. There's always the same number of pages, and we just filter out other messages. You can end up with page 1 being empty, page 2 having 1 error entry, etc.
SELECT post_id, meta_value
FROM wp_postmeta
WHERE meta_key = 'syn_log'
GROUP BY post_id
ORDER BY meta_id DESC
LIMIT 0, 100
@joshbetz
Copy link
Contributor Author

We also limit the number of logs that we store, so we should increase or remove that limit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants