I’m giving this a try tonight. Can’t wait. It looks great and I can think of a lot of people for whom it’d come really handy!
Thanks for this fantastic contribution. Have you used this version in production yet?
No but I’ve tested it quite well I hope. The first screenshot says 76 revisions and yes, I’ve saved that page 76 times. Hopefully that’s enough for a first version. The only possible danger should be if the rollback don’t work (but it should work just fine).
I expect it to be even more reliable with each new version.
I also have many things on my todolist that I will probably add in the next versions.
Examples of possible new features:
Rollback warning
Restore a just single field
Revision clean up buttons
Translations for other languages
Statistic information (filesize, number of positive/negative diff etc)
I renamed the folder from revisions to kirby-revisions right before uploading it to the repo yesterday but forgot to rename the file. So the plugin will probably not have worked at all.
Sorry about that!
2.1
Bugfix: Renamed revisions.php to kirby-revisions.php to match the folder name.
Renamed type to action.
I’m volunteering for the French version When can I start?
Probably not today. I think it’s important to nail the core features that I have in mind first. But great to have some French help!
Hi Jens
Thanks for this awesome plugin. I was waiting for something like that. Today I did a quick test, but I didn’t get it to work.
I used a fresh kirby 2.3.2 installation and installed revision manually, added the blueprint field and wanted to start.
But my panel page was empty, no fields shown at all. In my php log a found a fatal error: PHP Fatal error: Method RevisionsField::__toString() must not throw an exception in /Users/Sites/github/kirbytest/kirby/toolkit/lib/brick.php on line 0
I’m not so used to php so I can’t do anything with this error.
I’m using MAMP with PHP 7.
Thanks for your help and your great work for the kirby community.
Not a very good start but I’ve been hunting bugs this morning and we are now at version 2.4.
I tested it with a fresh starterkit with Kirby CLI with both multi language and without. Everything works like expected. Hopefully it will work for the rest of you as well.
Changelog
2.4
Bugfix: Fixed issue with rollback in multi language
2.3
Bugfix: Fixed an issue so it also work without multi language
2.2
Bugfix: Corrected an error in package.json
Bugfix: Added fix for pages that don’t have revisions yet
Feature: Added filesize to revisions fields
The reason I released so many versions was that I wanted to fix the major bugs quickly
What is a bit confusing form the users perspective is that the latest entry is actually the current version. So if I want to rollback one step I have to go the the second version.
Maybe we can hide the latest entry? Or am I thinking the wrong way?
I just wondered why the action is prominently displayed, even though in the screenshots it’s always the same one. Now I understand why I’ll have to use the plugin a bit before I make suggestions
@jonathanhess I want to involve at least one more person into this issue. Do you agree with my suggestion or @Malvese on this one? Other people can vote as well. Thanks!
I like both. But I prefere your version, where the current is not listed.
If you go for Malvese’s one, I would deactivate the link so its not clickable.
Hi Jens
I installed the plugin on a website where I use
c::set('date.handler', 'strftime');
Obviously this breaks the date in revisions. Is there a way to tell the plugin to use also strftime?
And a small improvement would be, if it would use the field label instead of the field name. On the detail page the titles are the fieldnames so that could be confusing. This should be changed to the blueprints label name.
See screenshot above. When visiting a revision in the panel there is a Revision info section. It shows template, action and diff.
Missing in blueprint warning
Below the fields diff in the revision there can be a warning if there are fields that are missing in the blueprint. This can be good to know about because these fields is probably not visible on the panel page, even if they are restored in the content file.