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

Merge pull request #3841 from wallabag/fix/bad-order-api

Fix bad order parameter in the API

Avoid error when a bad `order` parameter is given

Only allowed parameter are asc & desc

Come back to stable version

Previous one was used because some PHPUnit version where deleted and tests can't be run anymore.

Merge pull request #3837 from wallabag/2.3.7-dev

Jump to 2.3.7-dev

Jump to 2.3.7-dev

Merge pull request #3835 from wallabag/prepare-2.3.6

Prepare 2.3.6 release

Prepare 2.3.6 release

Merge pull request #3826 from wallabag/epub-toc

Rework of EPUB/PDF exports

Merge pull request #3831 from wallabag/fix/api-bad-client-id

Cast client id to avoid PG error

Merge pull request #3833 from techexo/patch-1

Fix settings field inverted

Update entries.html.twig

Should fix https://github.com/wallabag/wallabag/issues/3832

Update entries.html.twig

Should fix https://github.com/wallabag/wallabag/issues/3832

Cast client id to avoid PG error

If someone send a malformated client_id when trying to authenticate using the API we got a 500 if wallabag use postgres because the request send a string instead of an integer.

EntriesExport: avoid else on $authors

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

EntriesExport: sanitize filename and fix tests

Filename will now only use a-zA-Z0-9-' and space.

Fixes remaining filename issue on #3811

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

EntriesExport/pdf: move notice to the end, add metadata cover

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

Add translations

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

EntriesExport/epub: add metadata to each entry's cover

Add metadata to the cover of each entry:

- Publishers
- Estimated reading time
- Date of creation ("Added on")
- Address (URL)

Related to #2821

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

EntriesExport: change authors and title when not single entry export

Change '{method} authors' (which gives 'Tag_entries authors' when
exporting a tag) to 'Various authors'.

When exporting a tag (tag_entries), change the title from 'Tag_entries
articles' to 'Tag {tag} articles'.

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

EntriesExport/epub: revert c779373f, move exportinfo to the end of the book

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

EntriesExport/epub: use sha1 sums for filenames, fix and rename title chapters

This commit renames entry chapters file using a sha1 sum of their title
for simplicity. Also we fix the 'Title' chapter duplicate issue by using
the hash of the related entry and the suffix '_title'.

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

EntriesExport/epub: remove TOC page

This change only remove the rendered page of the TOC at the end of the
book, the TOC remains available to readers.

Fixes #3603

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

Merge pull request #3827 from wallabag/epub-quote

EntriesExport/epub: replace epub identifier with unique urn

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

EntriesExport/epub: replace epub identifier with unique urn

We replace the title used as the unique identifier of the epub file with
a urn following the format:

  urn:wallabag:{sha1("wallabagUrl:listOfEntryIdsSeparatedByComma")}

This format is repeatable: it always gives the same uid for the same
list of entries.

Fixes #3811

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

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 #3799 from wallabag/mruminski-patch-1

Update messages.pl.yml

Update messages.pl.yml

Add missing translation

Merge pull request #3758 from wallabag/dropping-php5

Dropping PHP < 7.1

Fix utf8mb4 on vendor tables

When creating the schema for test these tables use default length for
string: 255. Which fail when using utf8mb4.

> Syntax error or access violation: 1071 Specified key was too long; max key length is 767 bytes

Also move the `setKeepStaticConnections` in before and after class to
avoid:

> SAVEPOINT DOCTRINE2_SAVEPOINT_2 does not exist

See https://github.com/dmaicher/doctrine-test-bundle#troubleshooting

Use our own fork for CraueConfigBundle

Mostly to fix utf8mb4 issue

Force PHPUnit

Looks like "dama/doctrine-test-bundle" isn't compatible with PHPUnit 5.7
(required automatically by PHPUnit Bridge)

CS

Fix tests

Remove custom Postgres class

Because PG > 10 is now supported by DBAL >= 2.6.0

Update fixtures

Jump to unrelease predis

To fix deprecated message regarding `each()`

Cleanup

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>

Fix RulerZBundle

People should really follow semver and provide UPGRADE file when they
provide a library ...

Jump to PHP 7.1+

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

# Conflicts:
# web/wallassets/baggy.js
# web/wallassets/manifest.json
# web/wallassets/material.css
# web/wallassets/material.js

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>