| Age | Commit message (Collapse) | Author |
|
|
|
https://github.com/FreshRSS/FreshRSS/issues/711
https://github.com/FreshRSS/FreshRSS/pull/715
|
|
https://github.com/FreshRSS/FreshRSS/issues/711
|
|
Before, you could leave a configuration page without knowing if you saved your changes or not.
Now, there is an alert poping up if you have unsaved changes. It will ask you if you want to stay on the page and save your changes or leave the page and loose your changes.
See #739
|
|
Refactor exceptions
|
|
|
|
Change the name and messages
|
|
There is now an object to manipulate user queries. It allows to move logic to handle those from the view and the controller in the model.
Thus making the view and the controller easier to read.
I introduced a new interface to start using dependency injection. There is still some rough edges but we are moving in the right direction.
The new object is fully tested but it still need some improvements, for instance, it is still tied to the search object.
There might be a better way to do that.
|
|
As I do not have the time at the moment to verify the change of inheritance, I changed it back to the original.
|
|
I removed unnecessary constructors and unnecessary inheritance
|
|
|
|
|
|
In some part of the code, the search is displayed as is and crash since there is no way to display the search object as a string.
|
|
Before, the default value was undefined. Now it always has a value even when no value is provided.
|
|
It is now possible to combine multiple keywords to do a search. The separation of concern is better now since the search extraction is not done in the DAO anymore.
At the moment, a multiple keyword search is stored as this. It could be nice to have it rendered differently in the search page to make it more readable.
At the moment, there is a problem with search enclosed by ". Same search works well when enclosed by '.
|
|
The lib_date.php library was missing in the Search object file. It is required to make date conversion in that object.
|
|
Harmonize share configuration view.
|
|
I change the translation for the 3 languages. German has to be checked since I used Google translate to get the translation.
|
|
Before, for shares that don't need options, only a button to remove it was visible. It was source of confusion for users.
I changed the look of those shares by using the same layout as others (minus the help). As there is no configuration possible for the url, the field is disabled but it is possible to change the name of the share.
See #787
|
|
There were multiple calls made to the cleaning method that were unnecessary since it is useful only on the last call.
It allows to simplify code by returning values ealier.
|
|
|
|
I figured that the code for the search could be extracted from the context to have separation of concern.
It supports multiple keywords. It suports also multiple tag keywords.
|
|
This feature is not in use at the moment, but it will be handy to reorganize the query building process. It allows to have more than one keyword in the search box.
Full tests are available as well.
It probably needs a refactoring later, but I think this is the first step to make the application full object oriented and testable.
|
|
See https://github.com/FreshRSS/FreshRSS/issues/711
|
|
Temporary fix:
- Change syslog by Minz_Log::notice in most of the files
- Logs are stored in USERS_PATH/_/log.txt for actualize_script.php
- Simply comment syslog in SimplePie
See https://github.com/FreshRSS/FreshRSS/issues/711
|
|
UI was frustrating when input had the focus: we needed to click elsewhere to
be able to open the next dropdown.
See https://github.com/FreshRSS/FreshRSS/issues/777
|
|
|
|
FreshRSS should not be associated to me but to the community:
- Remove links to send me email or references to a "lead developer"
- Change old Github links marienfressinaud/FreshRSS to FreshRSS/FreshRSS
|
|
See https://github.com/FreshRSS/FreshRSS/issues/775
|
|
Related to https://github.com/FreshRSS/FreshRSS/commit/ec75139939780810f43f4e85fbf37de2409fe584
See https://github.com/FreshRSS/FreshRSS/issues/769
|
|
Fix https://github.com/FreshRSS/FreshRSS/issues/769
|
|
ealdraed-de-translation
|
|
https://github.com/FreshRSS/FreshRSS/commit/964a768bc384425e45ae953bff0f6f757078e7d6#commitcomment-9385695
|
|
This reverts commit 964a768bc384425e45ae953bff0f6f757078e7d6.
|
|
|
|
https://github.com/FreshRSS/FreshRSS/commit/211569ef85f50891035e3e2645ec0c87badec1e1#commitcomment-9384719
|
|
- lib_opml was not in its newest version
- FRSS supports OPML file without text attributes
Fix https://github.com/FreshRSS/FreshRSS/issues/758
|
|
Caused searches such as "intitle:&" to fail after paging, and
possible XSS vulnerabilities.
Discovered during https://github.com/FreshRSS/FreshRSS/issues/754
|
|
https://github.com/FreshRSS/FreshRSS/issues/759
Suggestion: the static objects should be user-independent (or at least
with the possibility to be re-set), while the FreshRSS object and its
attributes should be user-dependent.
|
|
Prevent JavaScript errors when the shortcuts have not been defined in
the config file.
https://github.com/FreshRSS/FreshRSS/issues/733
|
|
There were warnings when reading extensions (trying to use e.g. README
and .gitignore as directories), and when reading update file.
https://github.com/FreshRSS/FreshRSS/issues/733
|
|
mark_when was not taken in consideration.
|
|
- updateFeed() returns 0 if nothing changes so the test was false.
- Redirection in case of error is better now by redirecting on the right feed
Fix https://github.com/FreshRSS/FreshRSS/issues/755
|
|
notif_* messages have been moved into feedback array.
|
|
If the request fails:
- Open a notification to inform user
- Remove pending index from the pending_feeds list
Fix https://github.com/FreshRSS/FreshRSS/issues/751
|
|
- Error code and logs was not propagated from Minz_Error to the controller
- header was bad (200 instead of 404 or 403)
Related to https://github.com/FreshRSS/FreshRSS/issues/751
|
|
See https://github.com/FreshRSS/FreshRSS/issues/755
|
|
|
|
If default view is "global", the popup tried to get articles on the
default view which was… "global"! Articles are present on the "normal" view
instead.
|
|
A new shortcut was never saved because ConfigurationSetter never set a
shortcut which did not appear in the initial conf.
|