]> git.immae.eu Git - github/wallabag/wallabag.git/log
github/wallabag/wallabag.git
7 years agoValidate language & preview picture fields
Jeremy Benoist [Thu, 8 Jun 2017 19:51:46 +0000 (21:51 +0200)] 
Validate language & preview picture fields

Instead of saving the value of each field right into the content without any validation, it seems better to validate them.
This might sounds obvious now we say that.

7 years agoMerge pull request #3187 from wallabag/api-client-credentials
Jérémy Benoist [Thu, 8 Jun 2017 15:24:49 +0000 (17:24 +0200)] 
Merge pull request #3187 from wallabag/api-client-credentials

Create (and return) a client after creating a new user using the API

7 years agoAdd ability to name the client 3187/head
Jeremy Benoist [Thu, 8 Jun 2017 12:25:44 +0000 (14:25 +0200)] 
Add ability to name the client

7 years agoMerge pull request #3189 from wallabag/add-discourse
Nicolas Lœuillet [Thu, 8 Jun 2017 12:02:12 +0000 (14:02 +0200)] 
Merge pull request #3189 from wallabag/add-discourse

Added link to community in issue template

7 years agoAdded link to community in issue template 3189/head
Nicolas Lœuillet [Thu, 8 Jun 2017 11:34:14 +0000 (13:34 +0200)] 
Added link to community in issue template

7 years agoCS
Jeremy Benoist [Wed, 7 Jun 2017 21:23:34 +0000 (23:23 +0200)] 
CS

7 years agoCreate a client when creating a user using the api
Jeremy Benoist [Wed, 7 Jun 2017 21:23:28 +0000 (23:23 +0200)] 
Create a client when creating a user using the api

While creating a new user using the API, we also create a new client for the current user.
So the app which just create the user can use its newly created client to configure the app.

That new client is only return after creating the user.
When calling the endpoint /api/user to get user information, the new client information won’t be return.

7 years agoMerge pull request #3093 from aaa2000/annotation-error-on-save
Nicolas Lœuillet [Wed, 7 Jun 2017 14:30:27 +0000 (16:30 +0200)] 
Merge pull request #3093 from aaa2000/annotation-error-on-save

Displays an error with an annotation with a too long quote

7 years agoMerge pull request #3181 from wallabag/api-content-patch
Jérémy Benoist [Wed, 7 Jun 2017 13:40:59 +0000 (15:40 +0200)] 
Merge pull request #3181 from wallabag/api-content-patch

Add ability to patch an entry with more fields

7 years agoRefactorize the way to save an Entry in the API 3181/head
Jeremy Benoist [Wed, 7 Jun 2017 13:07:55 +0000 (15:07 +0200)] 
Refactorize the way to save an Entry in the API

7 years agoMerge pull request #3184 from wallabag/better-way-image-extension
Jérémy Benoist [Wed, 7 Jun 2017 11:41:25 +0000 (13:41 +0200)] 
Merge pull request #3184 from wallabag/better-way-image-extension

Use an alternative way to detect images

7 years agoMerge pull request #3179 from wallabag/show-user-command
Jérémy Benoist [Wed, 7 Jun 2017 07:40:56 +0000 (09:40 +0200)] 
Merge pull request #3179 from wallabag/show-user-command

Show user command

7 years agoFix postgresql migration 3093/head
adev [Tue, 6 Jun 2017 22:12:20 +0000 (00:12 +0200)] 
Fix postgresql migration

7 years agoMerge remote-tracking branch 'origin/master' into 2.3
Jeremy Benoist [Tue, 6 Jun 2017 18:15:21 +0000 (20:15 +0200)] 
Merge remote-tracking branch 'origin/master' into 2.3

7 years agoMerge pull request #3177 from wallabag/api-create-user-update
Jérémy Benoist [Tue, 6 Jun 2017 13:25:05 +0000 (15:25 +0200)] 
Merge pull request #3177 from wallabag/api-create-user-update

API user creation behing a toggle

7 years agoMerge pull request #3165 from wallabag/it-translation-update
Jérémy Benoist [Tue, 6 Jun 2017 09:44:18 +0000 (11:44 +0200)] 
Merge pull request #3165 from wallabag/it-translation-update

Update Italian translation

7 years agoUse an alternative way to detect image 3184/head
Jeremy Benoist [Mon, 5 Jun 2017 20:54:02 +0000 (22:54 +0200)] 
Use an alternative way to detect image

When parsing content to retrieve images to save locally, we only check for the content-type of the image response.
In some case, that value is empty.
Now we’re also checking for the first few bytes of the content as an alternative to detect if it’s an image wallabag can handle.
We might get higher image supports using that alternative method.

7 years agoFix review 3179/head
Thomas Citharel [Sun, 4 Jun 2017 15:57:40 +0000 (17:57 +0200)] 
Fix review

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
7 years agoDisplays an error with an annotation with a too long quote
adev [Sun, 7 May 2017 15:21:30 +0000 (17:21 +0200)] 
Displays an error with an annotation with a too long quote

Fix #2762

7 years agoMerge pull request #3180 from wallabag/mruminski-patch-1 3177/head
Jérémy Benoist [Fri, 2 Jun 2017 19:02:07 +0000 (21:02 +0200)] 
Merge pull request #3180 from wallabag/mruminski-patch-1

Update PL translation

7 years agoAdd ability to patch an entry with more fields
Jeremy Benoist [Fri, 2 Jun 2017 18:52:49 +0000 (20:52 +0200)] 
Add ability to patch an entry with more fields

Like when we create an entry, we can now patch an entry with new fields:
- content
- language
- preview_picture
- published_at
- authors

7 years agoReturn 201 on user creation
Jeremy Benoist [Fri, 2 Jun 2017 18:03:25 +0000 (20:03 +0200)] 
Return 201 on user creation

7 years agoUpdate PL transation 3180/head
Mateusz Rumiński [Fri, 2 Jun 2017 17:35:25 +0000 (19:35 +0200)] 
Update PL transation

add missing translation

7 years agoAdd show user command
Thomas Citharel [Fri, 2 Jun 2017 14:53:03 +0000 (16:53 +0200)] 
Add show user command

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
7 years agoAdd :it: translation
Jérémy Benoist [Fri, 2 Jun 2017 09:45:45 +0000 (11:45 +0200)] 
Add :it: translation

7 years agoUse a dedicated file for wallabag config
Jeremy Benoist [Fri, 2 Jun 2017 08:46:04 +0000 (10:46 +0200)] 
Use a dedicated file for wallabag config

7 years agoAdd translation
Jeremy Benoist [Fri, 2 Jun 2017 08:27:15 +0000 (10:27 +0200)] 
Add translation

7 years agoAPI user creation behing a toggle
Jeremy Benoist [Fri, 2 Jun 2017 08:19:33 +0000 (10:19 +0200)] 
API user creation behing a toggle

I've added a toggle feature (in internal settings) so that user api creation can be disabled while form registration still can be enabled.
Also, the /api/user endpoint shouldn't require authentication. Even if we check the authentication when sending a GET request, to retrieve current user information.

I've moved all the internal settings definition to config to avoid duplicated place to define them.
I don't know why we didn't did that earlier.

7 years agoMerge pull request #2708 from jcharaoui/import-disablecontentupdate
Jérémy Benoist [Fri, 2 Jun 2017 09:26:37 +0000 (11:26 +0200)] 
Merge pull request #2708 from jcharaoui/import-disablecontentupdate

Import disableContentUpdate

7 years agoUpdate messages.it.yml 3165/head
Jérémy Benoist [Fri, 2 Jun 2017 07:17:45 +0000 (09:17 +0200)] 
Update messages.it.yml

7 years agoMerge pull request #3176 from wallabag/fix-image-download
Nicolas Lœuillet [Fri, 2 Jun 2017 07:12:11 +0000 (09:12 +0200)] 
Merge pull request #3176 from wallabag/fix-image-download

Replace images with & in url

7 years agoCS 3176/head
Jeremy Benoist [Thu, 1 Jun 2017 20:58:38 +0000 (22:58 +0200)] 
CS

7 years agoReplace images with &
Jeremy Benoist [Thu, 1 Jun 2017 20:50:33 +0000 (22:50 +0200)] 
Replace images with &

Images with `&` in the path weren’t well replaced because they might be with `&amp;` in the html instead.

Replacing `&` with `&amp;` fix the problem.

7 years agoFix option attributes 2708/head
Jeremy Benoist [Thu, 1 Jun 2017 13:44:36 +0000 (15:44 +0200)] 
Fix option attributes

7 years agoMerge pull request #3171 from wallabag/drop-fucking-sqlite
Nicolas Lœuillet [Thu, 1 Jun 2017 12:46:01 +0000 (14:46 +0200)] 
Merge pull request #3171 from wallabag/drop-fucking-sqlite

Defined MySQL as the default rdbms for wallabag

7 years agoWe don't need that getter
Jeremy Benoist [Thu, 1 Jun 2017 10:46:07 +0000 (12:46 +0200)] 
We don't need that getter

7 years agoNo need to catch that Exception
Jeremy Benoist [Thu, 1 Jun 2017 09:45:02 +0000 (11:45 +0200)] 
No need to catch that Exception

7 years agoRewrote code & fix tests
Jeremy Benoist [Thu, 1 Jun 2017 09:31:45 +0000 (11:31 +0200)] 
Rewrote code & fix tests

7 years agoMerge pull request #3172 from wallabag/more-scruti
Jérémy Benoist [Thu, 1 Jun 2017 08:29:32 +0000 (10:29 +0200)] 
Merge pull request #3172 from wallabag/more-scruti

More Scrutinizer issues fixed

7 years agoCS
Jeremy Benoist [Thu, 1 Jun 2017 07:52:09 +0000 (09:52 +0200)] 
CS

7 years agoFix tests
Jeremy Benoist [Tue, 30 May 2017 15:48:24 +0000 (17:48 +0200)] 
Fix tests

7 years agoCS
Jeremy Benoist [Tue, 30 May 2017 14:21:25 +0000 (16:21 +0200)] 
CS

7 years agoReplace Wallabag v1 error strings with v2 strings
Jerome Charaoui [Fri, 16 Dec 2016 14:46:21 +0000 (09:46 -0500)] 
Replace Wallabag v1 error strings with v2 strings

7 years agoAdd disableContentUpdate import option
Jerome Charaoui [Wed, 7 Dec 2016 20:16:49 +0000 (15:16 -0500)] 
Add disableContentUpdate import option

This commit also decouples the "import" and "update" functions inside
ContentProxy. If a content array is available, it must be passed to the
new importEntry method.

7 years agoPut default fetching error title in global config
Jerome Charaoui [Wed, 7 Dec 2016 03:27:08 +0000 (22:27 -0500)] 
Put default fetching error title in global config

7 years agoAvoid returning objects passed by reference.
Jerome Charaoui [Wed, 7 Dec 2016 03:17:44 +0000 (22:17 -0500)] 
Avoid returning objects passed by reference.

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.

7 years agofixup! Set the right variable type 3172/head
Jeremy Benoist [Thu, 1 Jun 2017 07:36:01 +0000 (09:36 +0200)] 
fixup! Set the right variable type

7 years agoBoth parameter wasn't used in the function, removing them
Jeremy Benoist [Thu, 1 Jun 2017 07:30:38 +0000 (09:30 +0200)] 
Both parameter wasn't used in the function, removing them

7 years agoSet the right variable type
Jeremy Benoist [Thu, 1 Jun 2017 07:30:20 +0000 (09:30 +0200)] 
Set the right variable type

7 years agoPage parameter was never used in the function
Jeremy Benoist [Thu, 1 Jun 2017 07:29:16 +0000 (09:29 +0200)] 
Page parameter was never used in the function

It could have been used if we set the current page inside PreparePagerForEntries.
But we did that in each controller because we can have an OutOfRangeCurrentPageException

7 years agoChanged default database name 3171/head
Nicolas Lœuillet [Thu, 1 Jun 2017 07:17:05 +0000 (09:17 +0200)] 
Changed default database name

7 years agoMerge pull request #3137 from aaa2000/isolated-tests
Jérémy Benoist [Thu, 1 Jun 2017 05:58:17 +0000 (07:58 +0200)] 
Merge pull request #3137 from aaa2000/isolated-tests

Isolated tests

7 years agoFixed @tcitworld 's review
Nicolas Lœuillet [Wed, 31 May 2017 20:50:18 +0000 (22:50 +0200)] 
Fixed @tcitworld 's review

7 years agoDefined MySQL as the default rdbms for wallabag
Nicolas Lœuillet [Wed, 31 May 2017 20:36:21 +0000 (22:36 +0200)] 
Defined MySQL as the default rdbms for wallabag

7 years agoUpdated :it: translations
Nicolas Lœuillet [Wed, 31 May 2017 20:20:44 +0000 (22:20 +0200)] 
Updated :it: translations
Thank you @matteocoder

7 years agoMerge pull request #3077 from wallabag/add-tags-list-view
Nicolas Lœuillet [Wed, 31 May 2017 19:18:14 +0000 (21:18 +0200)] 
Merge pull request #3077 from wallabag/add-tags-list-view

Added tags on list view

7 years agoReduced border-radius for tags 3077/head
Nicolas Lœuillet [Wed, 31 May 2017 18:31:35 +0000 (20:31 +0200)] 
Reduced border-radius for tags

7 years agoMerge pull request #3106 from wallabag/api-content
Jérémy Benoist [Wed, 31 May 2017 15:26:05 +0000 (17:26 +0200)] 
Merge pull request #3106 from wallabag/api-content

Allow other fields to be send using API

7 years agoIsolated tests 3137/head
adev [Mon, 15 May 2017 18:47:59 +0000 (20:47 +0200)] 
Isolated tests

Use https://github.com/dmaicher/doctrine-test-bundle to have test isolation.

7 years agoAdd some flex magic
Thomas Citharel [Wed, 31 May 2017 12:34:46 +0000 (14:34 +0200)] 
Add some flex magic

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
7 years agoPR has been merged in graby 3106/head
Jérémy Benoist [Tue, 30 May 2017 16:11:55 +0000 (18:11 +0200)] 
PR has been merged in graby

7 years agoFix tests following rebase
Jeremy Benoist [Tue, 30 May 2017 08:08:50 +0000 (10:08 +0200)] 
Fix tests following rebase

7 years agoCS
Jeremy Benoist [Mon, 29 May 2017 08:14:01 +0000 (10:14 +0200)] 
CS

7 years agoForced date can now be a timestamp too
Jeremy Benoist [Wed, 24 May 2017 14:44:03 +0000 (16:44 +0200)] 
Forced date can now be a timestamp too

Add adding more tests for forced content

7 years agoImprove docs
Jeremy Benoist [Wed, 24 May 2017 14:02:49 +0000 (16:02 +0200)] 
Improve docs

7 years agoRemove htmlawed and use graby instead
Jeremy Benoist [Tue, 16 May 2017 21:11:20 +0000 (23:11 +0200)] 
Remove htmlawed and use graby instead

Instead of using htmlawed (which is already used in graby) use graby directly (which require some refacto on graby side).
Still needs some tests.

7 years agoReduce imported entries
Jeremy Benoist [Tue, 16 May 2017 19:17:10 +0000 (21:17 +0200)] 
Reduce imported entries

Down from 2.2 minutes to 1.39 minutes

7 years agoUse graby ContentExtractor to clean html
Jeremy Benoist [Fri, 12 May 2017 05:53:21 +0000 (07:53 +0200)] 
Use graby ContentExtractor to clean html

It might be better to re-use some graby functionalities to clean html instead of building a new system.

7 years agoAdd support for authors
Jeremy Benoist [Thu, 11 May 2017 18:10:22 +0000 (20:10 +0200)] 
Add support for authors

7 years agoAllow other fields to be send using API
Jeremy Benoist [Thu, 11 May 2017 06:14:29 +0000 (08:14 +0200)] 
Allow other fields to be send using API

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)

7 years agoFixed tags display on grid view
Nicolas Lœuillet [Mon, 29 May 2017 08:59:30 +0000 (10:59 +0200)] 
Fixed tags display on grid view

7 years agoMerge pull request #3168 from wallabag/instapaper-tags-import
Nicolas Lœuillet [Wed, 31 May 2017 10:32:09 +0000 (12:32 +0200)] 
Merge pull request #3168 from wallabag/instapaper-tags-import

Add support for tag in Instapaper import

7 years agoMerge pull request #3138 from Kdecherf/2835-tags
Nicolas Lœuillet [Wed, 31 May 2017 09:48:42 +0000 (11:48 +0200)] 
Merge pull request #3138 from Kdecherf/2835-tags

Ignore ActionMarkAsRead when removing tag from entry

7 years agoMerge pull request #3161 from wallabag/scrutinizer-2.3
Nicolas Lœuillet [Wed, 31 May 2017 09:46:41 +0000 (11:46 +0200)] 
Merge pull request #3161 from wallabag/scrutinizer-2.3

Fix some Scrutinizer issues

7 years agoEnsure retrieved data are fresh 3138/head
Jeremy Benoist [Wed, 31 May 2017 09:11:02 +0000 (11:11 +0200)] 
Ensure retrieved data are fresh

7 years agoCS & improve tags assertions 3168/head
Jeremy Benoist [Wed, 31 May 2017 08:38:15 +0000 (10:38 +0200)] 
CS & improve tags assertions

7 years agoAdd support for tag in Instapaper import
Jeremy Benoist [Wed, 31 May 2017 08:38:00 +0000 (10:38 +0200)] 
Add support for tag in Instapaper import

7 years agoMerge pull request #3143 from wallabag/fix-wllbg2-import
Nicolas Lœuillet [Wed, 31 May 2017 07:08:34 +0000 (09:08 +0200)] 
Merge pull request #3143 from wallabag/fix-wllbg2-import

Fixed is_starred for wallabag v2 import

7 years agoMerge pull request #3167 from wallabag/doc-github-template
Nicolas Lœuillet [Wed, 31 May 2017 07:06:21 +0000 (09:06 +0200)] 
Merge pull request #3167 from wallabag/doc-github-template

Fix link in Github template

7 years agoFix link in Github template 3167/head
Jérémy Benoist [Wed, 31 May 2017 05:12:55 +0000 (07:12 +0200)] 
Fix link in Github template

7 years agoTagController: ignore ActionMarkAsRead when removing tag from entry
Kevin Decherf [Sun, 21 May 2017 14:35:06 +0000 (16:35 +0200)] 
TagController: ignore ActionMarkAsRead when removing tag from entry

Fixes #2835

Signed-off-by: Kevin Decherf <kevin@kdecherf.com>
7 years agoAdd tests 3143/head
Jeremy Benoist [Tue, 30 May 2017 18:15:41 +0000 (20:15 +0200)] 
Add tests

7 years agoMerge pull request #3065 from wallabag/api-creation-endpoint
Jérémy Benoist [Tue, 30 May 2017 13:26:11 +0000 (15:26 +0200)] 
Merge pull request #3065 from wallabag/api-creation-endpoint

Register through API

7 years agoUpdate :it: translation
Jérémy Benoist [Tue, 30 May 2017 12:06:38 +0000 (14:06 +0200)] 
Update :it: translation

7 years agoCS 3161/head
Jeremy Benoist [Tue, 30 May 2017 11:01:25 +0000 (13:01 +0200)] 
CS

7 years agoUse a better way to set input for command
Jeremy Benoist [Tue, 30 May 2017 10:47:25 +0000 (12:47 +0200)] 
Use a better way to set input for command

Actually use the correct to way handle that

http://symfony.com/doc/current/components/console/helpers/questionhelper.html#testing-a-command-that-expects-input

7 years agoFix some Scrutinizer issues
Jeremy Benoist [Tue, 30 May 2017 09:39:15 +0000 (11:39 +0200)] 
Fix some Scrutinizer issues

7 years agoMerge remote-tracking branch 'origin/master' into 2.3
Jeremy Benoist [Tue, 30 May 2017 07:59:06 +0000 (09:59 +0200)] 
Merge remote-tracking branch 'origin/master' into 2.3

7 years agoAvoid side effect on other tests 3065/head
Jeremy Benoist [Tue, 30 May 2017 07:57:57 +0000 (09:57 +0200)] 
Avoid side effect on other tests

7 years agoAdded tags on list view
Nicolas Lœuillet [Wed, 3 May 2017 09:48:43 +0000 (11:48 +0200)] 
Added tags on list view

7 years agoMerge pull request #3147 from wallabag/delete-confirm
Jérémy Benoist [Tue, 30 May 2017 07:42:56 +0000 (09:42 +0200)] 
Merge pull request #3147 from wallabag/delete-confirm

Add some deletion confirmation to avoid mistake

7 years agoMerge pull request #3160 from wallabag/mruminski-patch-1 3147/head
Jérémy Benoist [Tue, 30 May 2017 07:10:54 +0000 (09:10 +0200)] 
Merge pull request #3160 from wallabag/mruminski-patch-1

Update messages.pl.yml

7 years agoRemove unknown validation_groups
Jeremy Benoist [Tue, 30 May 2017 05:56:41 +0000 (07:56 +0200)] 
Remove unknown validation_groups

The Profile validation_groups does not exist and then for validation to be skipped (like password length)

7 years agoRe-use `NewUserType` to validate registration
Jeremy Benoist [Tue, 30 May 2017 05:56:01 +0000 (07:56 +0200)] 
Re-use `NewUserType` to validate registration

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

7 years agoUpdate messages.pl.yml 3160/head
Mateusz Rumiński [Mon, 29 May 2017 21:47:00 +0000 (23:47 +0200)] 
Update messages.pl.yml

add missing translation

7 years agoupdate :de: messages.de.yml
Sven Fischer [Mon, 29 May 2017 19:10:00 +0000 (21:10 +0200)] 
update :de: messages.de.yml

7 years agoWIP
Thomas Citharel [Sat, 29 Apr 2017 17:22:50 +0000 (19:22 +0200)] 
WIP

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
7 years agoMerge pull request #3153 from wallabag/content-proxy-refactor
Thomas Citharel [Sun, 28 May 2017 19:50:23 +0000 (21:50 +0200)] 
Merge pull request #3153 from wallabag/content-proxy-refactor

Move Tags assigner to a separate file

7 years agoCS 3153/head
Thomas Citharel [Sun, 28 May 2017 10:59:48 +0000 (12:59 +0200)] 
CS

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
7 years agoCS
Thomas Citharel [Sat, 27 May 2017 23:16:01 +0000 (01:16 +0200)] 
CS

Signed-off-by: Thomas Citharel <tcit@tcit.fr>