Merge pull request #3820 from lizyn/bugfix/incorrect-calculation-of-CJK-characters-in-reading-time-estimation

Fix incorrect reading time calculation for entries with CJK characters

Counting two characters together as a word in CJK

Improve reading time tests

Merge pull request #3816 from wallabag/validate-import-entry

Validate imported entry to avoid error on import

Validate imported entry to avoid error on import

We got some imports with a missing `url` field generating some errors while trying to retrieve an existing entry with that url.
Introducing the `validateEntry` allow us to dismiss a message when it doesn't have an url (or other missing stuff in the future)

Merge pull request #3823 from wallabag/fix-tag-api-leak

Fix tag API leak

TagRestController: fix test for tag without entries

As the deletion now requires that at least one entry for the user must
be linked to the given tag, we fix the test testDeleteUserTag by linking
it to an entry.

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

TagRestController: rewrite delete actions to only retrieve tags related to the user

Fixes #3815

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

TagRestController: add tests to ensure that other user's tags are unreachable

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

TagRepository: refactor query builder for queries by userId

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

fixtures: refactor EntryData, TagData, add a new tag

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

fix incorrect reading time calculation for entries with CJK characters

Merge pull request #3814 from wallabag/2.3.6-dev

Jump to 2.3.6-dev and update release process

Jump to 2.3.6-dev and update release process

Fix release archive in `release.sh` and also fix a typo in the release process

Merge pull request #3812 from wallabag/prepare-2.3.5

Prepare 2.3.5 release

Prepare 2.3.5 release

Merge pull request #3810 from wallabag/remove-guzzle-site-authenticator-dev

No more dev for guzzle-site-authenticator

Merge pull request #3781 from wallabag/yarn-up

build: upgrade yarn dependencies, update prod assets

No more dev for guzzle-site-authenticator

build: upgrade yarn dependencies, update prod assets

Production assets are rebuilt against new dependency versions.

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

Merge pull request #3783 from wallabag/issue-3780

material: decrease size of tags on list view

Merge pull request #3782 from wallabag/issue-3779

material: fix missing thumbnail on list view

Merge pull request #3784 from giuppi/master

Fix Android app login issue

Merge pull request #3765 from burkemw3/patch-2

Remove preview picture from single entry view page for #1875

Merge pull request #3778 from wallabag/better-release

Jump to 2.3.5-dev and update release process

Fix Android app login issue

class and alt tags must be in the correct order for the android app to recognize the wallabag server

material: decrease size of tags on list view

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

material: fix missing thumbnail on list view

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

Jump to 2.3.5-dev and update release process

After the latest release, I update the documentation about it so it’s more clear

Remove preview picture from single entry view page for #1875

Showing the preview picture usually leads to showing a duplicate
image, and frequently leads to showing duplicate images directly
adjacent to each other.

Merge pull request #3775 from wallabag/prepare-2.3.4

Prepare 2.3.4 release

Create a custom swap to avoid error

Use latest bridge to avoid PHPUnit break

Because branch 4.8 & 5.7 have been removed

Use new Travis infra

Prepare 2.3.4 release

Merge pull request #3774 from Simounet/feature/card-ui

Card no preview replaced by wallabag logo

Card no preview replaced by wallabag logo

Merge pull request #3772 from techexo/patch-1

Update translation when marking as read

Update messages.en.yml

Update messages.fr.yml

Update messages.fr.yml

Modification pour éclaircir le sens de l'option

Merge pull request #3769 from burkemw3/patch-4

Add rel=noopener to target=_blank links

Add rel=noopener to target=_blank links

From https://developers.google.com/web/tools/lighthouse/audits/noopener:
- new pages will run in separate processes, avoiding any performance
  intermingling impacts
- new pages won't have access to current page's window object via JavaScript

Merge pull request #3766 from merwan/patch-1

Fix link to wallabag requirements in documentation

Fix link to wallabag requirements in documentation

Merge pull request #3764 from shtrom/fix-matches-notmatches-language

Fix documentation wording for matches/notmatches tagging rules

fixup! fixup! Fix documentation wording for matches/notmatches tagging rules

fixup! Fix documentation wording for matches/notmatches tagging rules

Fix documentation wording for matches/notmatches tagging rules

Signed-off-by: Olivier Mehani <shtrom@ssji.net>

Merge pull request #3763 from wallabag/ghpr-template-update

Add note on GitHub PR template to auto-close issues

misc: add note on GitHub PR template to auto-close issues

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

Merge pull request #3553 from wallabag/url-3529

Swap entry url with origin url if graby provides an updated one

phpcs

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

updateOriginUrl: remove 'query string' case from ignore list

Two urls with a different query string may refer to two different pages
so keep them both.

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

updateOriginUrl: add comment blocks for the parse_url diff check

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

updateOriginUrl: rewrite some if, resolving feedbacks from PR

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

ignoreOriginUrl: add initial support of ignore lists

Add the ability to specify hosts and patterns lists to ignore the given
entry url and replace it with the fetched content url without touching
to origin_url.

This initial support should be reworked in the following months to move
the hardcoded ignore lists in the database.

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

updateOriginUrl: add behavior when diff is fragment and query

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

Refactor updateOriginUrl to include new behaviors behaviors

- Leave origin_url unchanged if difference is an ending slash
- Leave origin_url unchanged if difference is scheme
- Ignore (noop) if difference is query string or fragment

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

ContentProxy: swap entry url to origin_url and set new url according to graby content

Closes #3529

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

ContentProxy: fix a corner case when entry.url is empty in updateEntry

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

Merge pull request #3733 from Deuchnord/root-abort-update

Abort running install and update script if root

Added argument to ignore the warning.

Merge pull request #3745 from burkemw3/patch-3

Add placeholder image to card-based gallery entries page

Add placeholder image to card-based gallery entries page for #3651

I referenced https://css-tricks.com/using-svg/ for using the SVG icon.
The icon has a black foreground and white background, which doesn't
look very good as a placeholder. So, using the background-image style
to show the svg and the inversion filter to make it white on gray.

With the image always present, there isn't much difference between
the with and without image preview templates, so I merged them.

Tested on Chrome, Firefox, and Safari on OSX.

Merge pull request #3606 from mritzmann/master

add manifest.json for android pwa

Merge pull request #3706 from shtrom/fix/gnu-make-bash

Makefile fixes for non GNU systems

Merge pull request #3743 from burkemw3/patch-1

Show tags on non-image gallery preview card

Merge pull request #3734 from davidmohamedfr/fix-dockerfile-php72

Fix dockerfile php72

Show tags on non-image gallery preview card

Tags and images aren't coupled, so they shouldn't be coupled in
the UI, either. This also makes the titles and source domains show
up consistently for image and non-image entry cards.

libpng12-dev was rename as libpng-dev

remove mcrypt since openssl is included in php 7.2

mcrypt extension doesn't exist anymore

Abort running install and update script if root (closes #3590)

Merge pull request #3725 from Tobi823/master

Bugfix: Sanitize the title of a saved webpage from invalid UTF-8 characters.

Run php-cs-fixer for fixing coding standard issues (on ContentProxyTest)

Run php-cs-fixer for fixing coding standard issues

Make helper methods strToHex and hexToStr in ContentProxyTest.php private
to prevent misusage (from outside this class)

Override the value of the given parameter ($title) with the (hopefully)
correct (to UTF-8) converted PDF title

Add tests for logic
Try to translate the title of a PDF from UTF-8 (then UTF-16BE, then WINDOWS-1252) to UTF-8

Try to detect the character encoding in PDFs and try to translate
the title from the PDF to UTF-8

Remove type declaration for PHP 5 compatibility

Bugfix: Sanitize the title of a saved webpage from invalid UTF-8 characters

Merge pull request #3726 from wallabag/fix-tests

Liberation goes https

Liberation goes https

Merge pull request #3719 from wallabag/fix-sort-parameters

Fix sort parameters

Updating node to 6.10

Same as defined in .travis.yml

Fix tests

Jump to the new Scrutinizer PHP analysis engine

Add missing parameters

Parameters weren’t passed to the sub function.

Merge pull request #3712 from wallabag/refactor-entry-queries

Rename getBuilderByUser and refactor query for untagged entries

Merge pull request #3717 from wallabag/php-72

PHP 7.2 shouldn't fail

PHP 7.2 shouldn't fail

php-cs-fixer

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

EntryRepository: refactor getBuilderForUntaggedByUser

Improve SQL performance by replacing size(e.tags) with a left join and a
null condition

Move the QueryBuilder logic into getRawBuilderForUntaggedByUser

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

EntryRepository: refactor getBuilderByUser

We refactor getBuilderByUser to separate QueryBuilder getter and the
orderBy(). The previous code of getBuilderByUser() has been moved to
getSortedQueryBuilderByUser(). getBuildByUser() now returns a
QueryBuilder without the call to orderBy().

A new method named sortQueryBuilder() returns a given QueryBuilder with
an orderBy() call using given sort parameters.

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

AnnotationRepository: rename getBuilderByUser

We rename getBuilderByUser to getSortedQueryBuilderByUser as long as the
method currently returns a QueryBuilder with an orderBy()

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

Merge pull request #3716 from wallabag/csfixer

php-cs-fixer: native_function_invocation

php-cs-fixer

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

Merge pull request #3707 from wallabag/fix-test-de-at

Removing failing test from Travis

Removing failing test from Travis