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

action.php: isset check for $meta['last_change']['date'] @ handle_recent #36

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

ErikCan
Copy link

@ErikCan ErikCan commented May 26, 2014

No description provided.

hh-lohmann and others added 5 commits May 21, 2014 10:02
solution: put the following line at the begin of the foreach loop starting in line 204:

`if ( !isset ( $ref ['_elem'] ) ) { break ; }`

See also https://www.dokuwiki.org/plugin:publish#error_when_using_the_old_revisions_view
solution: put the following line at the begin of the foreach loop starting in line 230 or 231 (inside "handle_recent"):

`if ( !isset ( $ref ['_elem'] ) ) { break ; }`
sorry, simple PHP break dos not do the full job, we need to check all `$ref` items in the `if` clause that sets `$member` BUT ONLY FOR THIS, not for the whole foreach body

See also https://www.dokuwiki.org/plugin:publish#error_when_using_the_old_revisions_view
…RRECTION

sorry, simple PHP break does not do the full job, we need to check all `$ref`, and that for the whole foreach body
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants