- error callback for loadHistory, to make the reload button enabled again after an error
- remove descriptionEdit's placeholder (for compatibilty with earlier versions of Qt)
- add placeholder text to descriptionEdit if supported by the user's version of Qt
- set contentEdit read-only while loading a new page
- saveButton was still showing "Saving" after saving had failed
- The 'save' button is disabled and shows 'No changes' at first. Editing contentEdit's text
enables the button and makes it show 'Save'.
- The 'save' button is disabled and shows 'Saving...' when the page is being saved, and stays
disabled until contentEdit's text has been changed again.
- The 'save' button shows 'Saved' once the page has been successfully saved. fixes#560
- contentEdit is readonly when the page is being saved
Most backends don't set it nb_messages and no application used it.
No backends set nb_unread and only one application tried to use it.
This should help avoid useless fillobj() calls in some cases.
fixes#827
There is now a "latest" collection (or "latest_nsfw").
The feature didn't look much used, since it didn't work on many
backends.
Using collections will make it easy to support other things
like most viewed, featured, etc.
As a bonus, I added tests for every backend with the feature.
I used autopep8 on some files and did carefully check the changes.
I ignored E501,E302,E231,E225,E222,E221,E241,E203 in my search, and at
least E501 on any autopep8 run.
Other style fixes not related to PEP8:
* Only use new-style classes. I don't think the usage of old-style
classes was voluntary. Old-style classes are removed in Python 3.
* Convert an if/else to a one-liner in mediawiki, change docstring style
change to a comment something that wasn't really appropriate for a
docstring.
* Unneeded first if condition in meteofrance
* Remove callbacks in Collection object
Make Collection a "dumb" object (and also a base object,
though it isn't very useful for now)
* Rename Path to WorkingPath, because it is more about managing state
than being a single path.
* Rewrite almost all WorkingPath, because the code was overly
complicated for no reason (I tried some special cases and it turned
out that fromstring didn't handle them, and that the
quote-escape-unquote was just unecessary). I also rewrote it to be
more pythonic (no more lambdas and maps) and added tests.
* Require the full split path when creating a Collection. Because, come to
think of it, an object needs an unique identifier; in the case of
Collections, it is the full path, not only its last part.
I might even replace the id by the full split path in the future.
* There is now only one way to get items of a Collection: calling
iter_resources().
* Rewrite flatten_resources to iter_resources_flat(), which just calls
iter_resources() recursively.
* Rewrite the collection part of the canalplus module. There is no more
callback or a page calling the browser to check another page!
The logic is only in iter_resources().
The resulting code is not very pretty, but it should get better.
As a bonus, avoid to reload the main XML file when we already have it
open.
* change_path() now expects a split path and not a string.
* up/home special cases for "cd" are handled in the same place, and
store the previous place properly (but are not yet exploitable by
an user command).
This is a big commit but it would be hard to split it in *working*
commits.
If you read this entire commit message, I will buy you a beer.
refs #774fixes#773
