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.

Merge pull request #2708 from jcharaoui/import-disablecontentupdate

Import disableContentUpdate

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 `&` in the html instead.

Replacing `&` with `&` 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

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

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

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

update :de: messages.de.yml

WIP

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

Merge pull request #3153 from wallabag/content-proxy-refactor

Move Tags assigner to a separate file

CS

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

CS

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

Move Tags assigner to a separate file

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

Add some deletion confirmation to avoid mistake

Fixed is_starred for wallabag v2 import

Merge pull request #3145 from wallabag/fix-so

Fix SO tests

Merge pull request #3146 from DamienCassou/fix-doc-url

Fix documentation URL for "How can I help to fix that?"

Fix documentation URL for "How can I help to fix that?"

Fix SO tests

SO switched to http yesterday

Switch back to latest guzzle-site-authenticator

Merge pull request #3133 from wallabag/testing-travis-failure

Let's play !

Ensure download_images_enabled is disabled

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)

Fold migrations & fixtures command on Travis

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

Merge pull request #3130 from X-dark/master

unify Download/Export wording.

modify key name as well

unify Download/Export wording.

This fixes #2967

Update of the oc version of CoreBundle

I hope I didn't break everything with the ' and ".

Merge pull request #3125 from wallabag/prepare-222

Prepare wallabag 2.2.3

Prepare wallabag 2.2.3

Merge pull request #3122 from wallabag/delete-doc

Removed embedded documentation

Merge pull request #3117 from wallabag/fix-doc

Fixed documentation URL

Fixed documentation URL

Merge pull request #3124 from wallabag/lock-guzzle-auth

Lock guzzle-site-authenticator to avoid errors

Lock guzzle-site-authenticator to avoid error

Merge pull request #3123 from X-dark/master

reorder contrib strings in about page

reorder contrib strings in about page

this fixes #2891

Removed embedded documentation

Merge pull request #3120 from aaa2000/fix-delete-annotation-user

Fix delete annotation when username is defined

Merge pull request #3113 from wallabag/upgrade-craue

Upgraded CraueConfigBundle to 2.0

Fix delete annotation when username is defined

When an user set his name in "Config" > "User Information", he
can't delete or update an annotation

Fix #2382

Upgraded CraueConfigBundle to 2.0

Merge pull request #3110 from wallabag/add-logger-matching-rules

Added logger when we match Tagging rules

Added tests on logs records