Cleanup test

Looks like we didn't ALWAYS get a value for language from 20minutes.
Ahem.

Convert - to _ in language

Mostly to increase language supports

Use Locale instead of Language

Fix test

de_DE is not valid language.
Zataz doesn’t send a valid language in their content (they use `fr-FR`).

CS

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.

Merge pull request #3187 from wallabag/api-client-credentials

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

Add ability to name the client

Merge pull request #3189 from wallabag/add-discourse

Added link to community in issue template

Added link to community in issue template

CS

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.

Merge pull request #3093 from aaa2000/annotation-error-on-save

Displays an error with an annotation with a too long quote

Merge pull request #3181 from wallabag/api-content-patch

Add ability to patch an entry with more fields

Refactorize the way to save an Entry in the API

Merge pull request #3184 from wallabag/better-way-image-extension

Use an alternative way to detect images

Merge pull request #3179 from wallabag/show-user-command

Show user command

Fix postgresql migration

Merge remote-tracking branch 'origin/master' into 2.3

Merge pull request #3177 from wallabag/api-create-user-update

API user creation behing a toggle

Merge pull request #3165 from wallabag/it-translation-update

Update Italian translation

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.

Fix review

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

Displays an error with an annotation with a too long quote

Fix #2762

Merge pull request #3180 from wallabag/mruminski-patch-1

Update PL translation

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

Return 201 on user creation

Update PL transation

add missing translation

Add show user command

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

Add :it: translation

Use a dedicated file for wallabag config

Add translation

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.

Merge pull request #2708 from jcharaoui/import-disablecontentupdate

Import disableContentUpdate

Update messages.it.yml

Merge pull request #3176 from wallabag/fix-image-download

Replace images with & in url

CS

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.

Fix option attributes

Merge pull request #3171 from wallabag/drop-fucking-sqlite

Defined MySQL as the default rdbms for wallabag

We don't need that getter

No need to catch that Exception

Rewrote code & fix tests

Merge pull request #3172 from wallabag/more-scruti

More Scrutinizer issues fixed

CS

Fix tests

CS

Replace Wallabag v1 error strings with v2 strings

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.

Put default fetching error title in global config

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.

fixup! Set the right variable type

Both parameter wasn't used in the function, removing them

Set the right variable type

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

Changed default database name

Merge pull request #3137 from aaa2000/isolated-tests

Isolated tests

Fixed @tcitworld 's review

Defined MySQL as the default rdbms for wallabag

Updated :it: translations
Thank you @matteocoder

Merge pull request #3077 from wallabag/add-tags-list-view

Added tags on list view

Reduced border-radius for tags

Merge pull request #3106 from wallabag/api-content

Allow other fields to be send using API

Isolated tests

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

Add some flex magic

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

PR has been merged in graby

Fix tests following rebase

CS

Forced date can now be a timestamp too

Add adding more tests for forced content

Improve docs

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.

Reduce imported entries

Down from 2.2 minutes to 1.39 minutes

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.

Add support for authors

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)

Fixed tags display on grid view

Merge pull request #3168 from wallabag/instapaper-tags-import

Add support for tag in Instapaper import

Merge pull request #3138 from Kdecherf/2835-tags

Ignore ActionMarkAsRead when removing tag from entry

Merge pull request #3161 from wallabag/scrutinizer-2.3

Fix some Scrutinizer issues

Ensure retrieved data are fresh

CS & improve tags assertions

Add support for tag in Instapaper import

Merge pull request #3143 from wallabag/fix-wllbg2-import

Fixed is_starred for wallabag v2 import

Merge pull request #3167 from wallabag/doc-github-template

Fix link in Github template

Fix link in Github template

TagController: ignore ActionMarkAsRead when removing tag from entry

Fixes #2835

Signed-off-by: Kevin Decherf <kevin@kdecherf.com>

Add tests

Merge pull request #3065 from wallabag/api-creation-endpoint

Register through API

Update :it: translation

CS

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

Fix some Scrutinizer issues

Merge remote-tracking branch 'origin/master' into 2.3

Avoid side effect on other tests

Added tags on list view

Merge pull request #3147 from wallabag/delete-confirm

Add some deletion confirmation to avoid mistake

Merge pull request #3160 from wallabag/mruminski-patch-1

Update messages.pl.yml

Remove unknown validation_groups

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

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

Update messages.pl.yml

add missing translation