| Age | Commit message (Collapse) | Author |
|---|
|
|
|
Add an unsaved changes alert on config pages
|
|
|
|
Can simplify the regex (faster because anchored) for cleaning URLs based
on the fact that we only have to deal with HTTP or HTTPS
https://github.com/FreshRSS/FreshRSS/pull/715
|
|
|
|
This reverts commit 1c38e646c3223571988b33e6a0d481426e3b6931.
|
|
|
|
Can simplify the regex (faster because anchored) for cleaning URLs based
on the fact that we only have to deal with HTTP or HTTPS
|
|
|
|
https://github.com/FreshRSS/FreshRSS/issues/711
https://github.com/FreshRSS/FreshRSS/pull/715
|
|
|
|
https://github.com/FreshRSS/FreshRSS/issues/711
|
|
https://github.com/FreshRSS/FreshRSS/issues/754
|
|
https://github.com/FreshRSS/FreshRSS/issues/754
Needs to check with many feeds to see if this does not introduce
incompatibilities with some valid feeds.
|
|
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.
|
|
Fix entry DAO query usage
|
|
I removed unnecessary constructors and unnecessary inheritance
|
|
I did not fix the call in the previous commit. I hope this one is the last change needed.
We definitely need a templating engine so we could use the same controller to output different things.
This will remove code duplication between the api and the web interface.
It will allows us to build other type of api, and also refactor the rss feed as a different view of the same dataset.
|
|
Add missing white space
|
|
Fix API to use the search object
|
|
Since the internal of the listWhere method was changed, the API wasn't working. It was still calling the method with the old parameters.
I didn't test it but now, it should work.
|
|
|
|
|
|
|
|
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.
|
|
Contributing file
|
|
Remove unnecessary sentences in "Contribute to documentation"
See https://github.com/FreshRSS/FreshRSS/issues/783
|
|
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.
|
|
|
|
Add a section about mailing lists
See https://github.com/FreshRSS/FreshRSS/issues/783
|
|
- remove TODO
- simplify info about "fix a bug" (get from https://github.com/github/gitignore
README)
- add few info
See https://github.com/FreshRSS/FreshRSS/issues/783
|
|
Add a cleaning process in the auto-remove feature.
|
|
Extract search from context
|
|
Before, when the status change was too quick, sometimes, articles weren't removed from the view.
Now they are removed by adding a statement that grabs past articles with specific properties and remove them.
See #738
|
|
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/783
|
Alexandre Alapetite
Alexis Degrugillier
Marien Fressinaud
Alexis Degrugillier
