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

Compatibility issue with Release 2016-06-26 "Elenor of Tsort" #72

Closed
jkldgoefgkljefogeg opened this issue Jun 28, 2016 · 8 comments
Closed
Labels

Comments

@jkldgoefgkljefogeg
Copy link

jkldgoefgkljefogeg commented Jun 28, 2016

doku.php?do=admin

Warning: Declaration of admin_plugin_move::getMenuText() should be compatible with DokuWiki_Admin_Plugin::getMenuText($language) in dokuwiki/lib/plugins/move/admin.php on line 276

@jkldgoefgkljefogeg
Copy link
Author

jkldgoefgkljefogeg commented Jun 28, 2016

functionality broken, when I tried to move a page I got, page not moved

Fatal error: Call to undefined function getRevisions() in lib/plugins/move/helper.php on line 402

@TheTomcat14
Copy link

This bug seems to be caused by the removal of deprecated code from the dokuwiki codebase.
(reference to the commit dokuwiki/dokuwiki@aa13c9a)

As a quick & dirty fix I added the code for getRevisions() to the file lib/plugins/move/helper.php (at the end of the file, outside the class declaration).
Adding the deprecated code to the plugin "should" protect the functionality from future dokuwiki updates.

A permanent/better solution would be to fix the 2 getRevisions() calls made in the plugin.

Code pasted at the very end of the file lib/plugins/move/helper.php

function getRevisions($id, $first, $num, $chunk_size = 8192, $media = false) { dbg_deprecated('class PageChangeLog or class MediaChangelog'); if($media) { $changelog = new MediaChangeLog($id, $chunk_size); } else { $changelog = new PageChangeLog($id, $chunk_size); } return $changelog->getRevisions($first, $num); }

@micgro42
Copy link
Collaborator

Your version of the plugin is outdated. Please update the move plugin to the current version.

@jkldgoefgkljefogeg
Copy link
Author

How did you determine the version I have is outdated? Did you read the comment above?

@micgro42
Copy link
Collaborator

lib/plugins/move/admin.php and lib/plugins/move/helper.php do not exist in the current version of the plugin.

@jkldgoefgkljefogeg
Copy link
Author

I am confused. These files are not in 2015-10-18 tag either. And the version from dokuwiki store has never been updated?
plugin.info.txt can probably get some update.

@jkldgoefgkljefogeg
Copy link
Author

There seems to be some issue with the dokuwiki extension manager. I reinstalled this extension but there were still files modified in 2014 (admin.php, helper.php, even .gitignore). Uninstall and install the plugin solved the problem. Sorry for the noise.

@micgro42
Copy link
Collaborator

It seems one must explicitly tell the extension manger to remove those old files. The linked pull request should solve this problem in the future.

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

No branches or pull requests

3 participants