-
Notifications
You must be signed in to change notification settings - Fork 35
Added the ability to rebuild the intire database from the admin area #82
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
base: master
Are you sure you want to change the base?
Conversation
What happened to helper.php? The whole file is changed? Did you change line endings or something? Why the code in action.php? This should be part of the admin component's handle() method. Can you please fix these two things? |
Hi, thanks for the feedback, very valid points. Although I haven’t noticed the change in line-endings, I always struggle with cross platform collaboration. I’ll login to my linux box and fix it on there. I found your guidelines for writing admin plugins, so this would probably be the time that I confess that I should have RTFM :) I’ll revert the commit, fix the changes and re-submit Cheers, Stephan From: Andreas Gohr [mailto:notifications@github.com] What happened to helper.php? The whole file is changed? Did you change line endings or something? Why the code in action.php? This should be part of the admin component's handle() method. Can you please fix these two things? — |
Hi, thanks for the rebuild functionality! We lost our database the other day and had to restore from a backup which missed the latest entries. Rebuilding added them back. I had to fix an issue with page titles, though, and addressed the points by splitbrain at the same time. Do I have to fork the plugin to provide a patch or is there another way? Andreas |
Hi Andreas, Sorry for the late reply. Yeah, I has the same issue. We do continuous deployment for our content, so we actually blow away all our data on a daily basis :) It’s been a while but I remember sending a pull request ages ago and did mean to follow up on the review comments before resubmitting. There are basically two approaches and depending on your technical skills, you can pick either of the two:
Whatever you think is the best for you. I’m happy to commit to follow up on this sooner than responding to this email. :) Thanks for contributing! Stephan From: andyboeh [mailto:notifications@github.com] Hi, thanks for the rebuild functionality! We lost our database the other day and had to restore from a backup which missed the latest entries. Rebuilding added them back. I had to fix an issue with page titles, though, and addressed the points by splitbrain at the same time. Do I have to fork the plugin to provide a patch or is there another way? Andreas — |
Does this have any advantage over the more general https://www.dokuwiki.org/plugin:searchindex? See also #27: Is it faster to rebuild data's database than DokuWiki's (search-) index? Both have to check all pages, or do I miss something? Note: a great feature for both ways were the ability to constrain rebuilding on a certain namespace (some namespaces may be of more interest for up-to-date indexes / databases than others), maybe (also) the other way round by excluding certain namespaces - for some namespaces "access by linking" does the job, while for others "access by searching / data plugin" is more important, and rebuilding only for the latter could be done much faster than rebuilding for all. |
No comments... Just awesome code :-)
Love your work Andreas!!