Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Avoid returning objects passed by reference. | Jerome Charaoui | 2017-06-01 | 1 | -11/+19 |
| | | | | | | Objects are always passed by reference, so it doesn't make sense to return an object which is passed by reference as it will always be the same object. This change makes the code a bit more readable. | ||||
* | Merge pull request #3137 from aaa2000/isolated-tests | Jérémy Benoist | 2017-06-01 | 12 | -262/+429 |
|\ | | | | | Isolated tests | ||||
| * | Isolated tests | adev | 2017-05-31 | 13 | -265/+433 |
| | | | | | | | | Use https://github.com/dmaicher/doctrine-test-bundle to have test isolation. | ||||
* | | Fix tests following rebase | Jeremy Benoist | 2017-05-31 | 1 | -2/+2 |
| | | |||||
* | | CS | Jeremy Benoist | 2017-05-31 | 1 | -3/+3 |
| | | |||||
* | | Forced date can now be a timestamp too | Jeremy Benoist | 2017-05-31 | 1 | -0/+81 |
| | | | | | | | | Add adding more tests for forced content | ||||
* | | Remove htmlawed and use graby instead | Jeremy Benoist | 2017-05-31 | 1 | -13/+13 |
| | | | | | | | | | | Instead of using htmlawed (which is already used in graby) use graby directly (which require some refacto on graby side). Still needs some tests. | ||||
* | | Reduce imported entries | Jeremy Benoist | 2017-05-31 | 8 | -491/+23 |
| | | | | | | | | Down from 2.2 minutes to 1.39 minutes | ||||
* | | Use graby ContentExtractor to clean html | Jeremy Benoist | 2017-05-31 | 1 | -0/+55 |
| | | | | | | | | It might be better to re-use some graby functionalities to clean html instead of building a new system. | ||||
* | | Add support for authors | Jeremy Benoist | 2017-05-31 | 1 | -0/+4 |
| | | |||||
* | | Allow other fields to be send using API | Jeremy Benoist | 2017-05-31 | 5 | -21/+20 |
|/ | | | | | | | | | | | | | | | | Entry API can now have these new fields: - content - language - preview_picture - published_at Re-use the ContentProxy to be able to do the same using the web UI (in the future). htmLawed is used to clean stuff from content, I hope it’ll be enough to avoid security breach. Lower content validation when we want to update an entry with content already defined. Before, language & content_type were required. If there weren’t provided, we re-fetched the content using graby. I think these fields aren’t required for an entry to be created. So I removed them. Which means some import from the v1 export won’t be re-fetched since they provide content, url & title. Also, remove liberation link from Readability import to avoid overlaping import (from wallabag v1, which had the same link) | ||||
* | Merge pull request #3168 from wallabag/instapaper-tags-import | Nicolas Lœuillet | 2017-05-31 | 12 | -31/+77 |
|\ | | | | | Add support for tag in Instapaper import | ||||
| * | CS & improve tags assertions | Jeremy Benoist | 2017-05-31 | 9 | -18/+34 |
| | | |||||
| * | Add support for tag in Instapaper import | Jeremy Benoist | 2017-05-31 | 3 | -13/+43 |
| | | |||||
* | | Merge pull request #3138 from Kdecherf/2835-tags | Nicolas Lœuillet | 2017-05-31 | 2 | -0/+28 |
|\ \ | | | | | | | Ignore ActionMarkAsRead when removing tag from entry | ||||
| * | | Ensure retrieved data are fresh | Jeremy Benoist | 2017-05-31 | 1 | -0/+6 |
| | | | |||||
| * | | TagController: ignore ActionMarkAsRead when removing tag from entry | Kevin Decherf | 2017-05-31 | 2 | -0/+22 |
| | | | | | | | | | | | | | | | | | | Fixes #2835 Signed-off-by: Kevin Decherf <kevin@kdecherf.com> | ||||
* | | | Merge pull request #3161 from wallabag/scrutinizer-2.3 | Nicolas Lœuillet | 2017-05-31 | 1 | -76/+30 |
|\ \ \ | |_|/ |/| | | Fix some Scrutinizer issues | ||||
| * | | Use a better way to set input for command | Jeremy Benoist | 2017-05-30 | 1 | -76/+30 |
| | | | | | | | | | | | | | | | | | | Actually use the correct to way handle that http://symfony.com/doc/current/components/console/helpers/questionhelper.html#testing-a-command-that-expects-input | ||||
* | | | Merge pull request #3143 from wallabag/fix-wllbg2-import | Nicolas Lœuillet | 2017-05-31 | 2 | -1/+2 |
|\ \ \ | |_|/ |/| | | Fixed is_starred for wallabag v2 import | ||||
| * | | Add tests | Jeremy Benoist | 2017-05-30 | 2 | -1/+2 |
| | | | |||||
* | | | Merge pull request #3065 from wallabag/api-creation-endpoint | Jérémy Benoist | 2017-05-30 | 3 | -3/+113 |
|\ \ \ | |_|/ |/| | | Register through API | ||||
| * | | Avoid side effect on other tests | Jeremy Benoist | 2017-05-30 | 1 | -1/+13 |
| | | | |||||
| * | | Remove unknown validation_groups | Jeremy Benoist | 2017-05-30 | 1 | -2/+2 |
| | | | | | | | | | | | | The Profile validation_groups does not exist and then for validation to be skipped (like password length) | ||||
| * | | Re-use `NewUserType` to validate registration | Jeremy Benoist | 2017-05-30 | 2 | -1/+99 |
| | | | | | | | | | | | | | | | | | | The only ugly things is how we handle error by generating the view and then parse the content to retrieve all errors… Fix exposition fields in User entity | ||||
* | | | Merge remote-tracking branch 'origin/master' into 2.3 | Jeremy Benoist | 2017-05-30 | 1 | -1/+1 |
|\ \ \ | |/ / |/| | | |||||
| * | | Fix documentation URL for "How can I help to fix that?" | Damien Cassou | 2017-05-24 | 1 | -1/+1 |
| | | | |||||
* | | | CS | Thomas Citharel | 2017-05-28 | 1 | -6/+0 |
| | | | | | | | | | | | | Signed-off-by: Thomas Citharel <tcit@tcit.fr> | ||||
* | | | Move Tags assigner to a separate file | Thomas Citharel | 2017-05-27 | 9 | -136/+169 |
| |/ |/| | | | | | Signed-off-by: Thomas Citharel <tcit@tcit.fr> | ||||
* | | Fix SO tests | Jeremy Benoist | 2017-05-24 | 1 | -4/+4 |
| | | | | | | | | SO switched to http yesterday | ||||
* | | Ensure download_images_enabled is disabled | Jeremy Benoist | 2017-05-19 | 1 | -1/+20 |
| | | | | | | | | | | Even if the tests fail, that config must stay disabled after the test. Otherwise it might timeout on other test (because it'll try to save all other images) | ||||
* | | Merge remote-tracking branch 'origin/master' into 2.3 | Jeremy Benoist | 2017-05-19 | 2 | -0/+54 |
|\| | |||||
| * | Merge pull request #3095 from aaa2000/api-error-on-fail-fetch-content | Thomas Citharel | 2017-05-10 | 1 | -0/+34 |
| |\ | | | | | | | Create a new entry via API even when its content can't be retrieved | ||||
| | * | Remove the created entry to avoid side effects on other tests | adev | 2017-05-09 | 1 | -8/+18 |
| | | | |||||
| | * | Create a new entry via API even when its content can't be retrieved | adev | 2017-05-08 | 1 | -0/+24 |
| | | | | | | | | | | | | Fix #2988 | ||||
| * | | Merge pull request #3076 from wallabag/hotfix/2979 | Nicolas Lœuillet | 2017-05-09 | 1 | -0/+20 |
| |\ \ | | | | | | | | | Disable negative numbers in filters | ||||
| | * | | Add some tests | Jeremy Benoist | 2017-05-09 | 1 | -0/+20 |
| | |/ | |||||
* | | | Added tests on logs records | Nicolas Lœuillet | 2017-05-12 | 1 | -3/+22 |
| | | | |||||
* | | | Fixed tests | Nicolas Lœuillet | 2017-05-12 | 1 | -1/+7 |
| | | | |||||
* | | | Replaced json_array with array | Nicolas Lœuillet | 2017-05-11 | 1 | -1/+1 |
| | | | | | | | | | | | | And fixed failing test due to @j0k3r :trollface: | ||||
* | | | Added headers field in Entry | Nicolas Lœuillet | 2017-05-11 | 1 | -0/+1 |
| | | | |||||
* | | | Merge pull request #3101 from wallabag/credentials-new-extrafields | Jérémy Benoist | 2017-05-10 | 1 | -13/+27 |
|\ \ \ | | | | | | | | | Skip auth when no credentials are found | ||||
| * | | | Skip auth when no credentials are found | Jeremy Benoist | 2017-05-09 | 1 | -13/+27 |
| | | | | | | | | | | | | | | | | If we can’t find a credential for the current host, even if it required login, we won’t add them and website will be fetched without any login. | ||||
* | | | | Retrieve tag / tags value from query or request | Jeremy Benoist | 2017-05-09 | 1 | -4/+26 |
|/ / / | | | | | | | | | | | | | It allows to request to delete a tag using query string instead of body parameter (which seems to be the standard). Instead of breaking the previous behavior, I used a generic way to retrieve parameter (which looks into request attributes, query parameters and request parameters) | ||||
* | | | Merge pull request #2751 from bdunogier/2.2-guzzle_subscribers_improvement | Jérémy Benoist | 2017-05-09 | 1 | -1/+1 |
|\ \ \ | | | | | | | | | Improved Guzzle subscribers extensibility | ||||
| * | | | Changed parsing of login_extra_fields in guzzle auth | Bertrand Dunogier | 2017-05-04 | 1 | -1/+1 |
| | | | | |||||
* | | | | Merge remote-tracking branch 'origin/master' into 2.3 | Jeremy Benoist | 2017-05-09 | 1 | -0/+16 |
|\ \ \ \ | | |/ / | |/| | | |||||
| * | | | Fix API pagination is broken if perPage is custom value | adev | 2017-05-08 | 1 | -0/+16 |
| | |/ | |/| | | | | | | | Fix #2720 | ||||
* | | | Merge pull request #3088 from wallabag/execute-migrations-after-install | Jérémy Benoist | 2017-05-05 | 2 | -5/+10 |
|\ \ \ | | | | | | | | | Added migrations execution after fresh install | ||||
| * | | | Added migrations execution after fresh install | Nicolas Lœuillet | 2017-05-05 | 4 | -7/+12 |
| | | | |