Merge pull request #3982 from wallabag/fix/https-test

Fix http/https test

Fix test

Merge pull request #3966 from wallabag/prepare-2.3.8

Prepare 2.3.8 release

Prepare 2.3.8 release

Merge pull request #3964 from wallabag/fix/intl

Fix Intl Locale issue

Fix Intl Locale issue

Some issue appeared after the release of PHP 7.2.17 about Intl Memory
Leak / infinite loop.
To fix it we should upgrade to Symfony 3.4 (which is done in wallabag
2.4) but for the 2.3 branch, we'll use a temporary fix for the Locale
issue.

Merge pull request #3922 from burkemw3/patch-5

Remove preview picture from share view page for #1875

Merge pull request #3943 from wallabag/img-referrer

Enable no-referrer on img tags, enable strict-origin-when-cross-origin by default

Remove preview picture from share view page for #1875

Essentially, same as commit 038fccd for single entry views. From that commit:
> Showing the preview picture usually leads to showing a duplicate
> image, and frequently leads to showing duplicate images directly
> adjacent to each other.

assets: regenerate material.css

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

Enable no-referrer on img tags, enable strict-origin-when-cross-origin by default

Fixes #3889

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

Merge pull request #3942 from wallabag/issue-3838

material: add metadata to list view

material: add metadata to list view

Add reading time and creation date to rows of list view.
Refactor styles using a sass mixin.

Fixes #3838

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

Merge pull request #3937 from wallabag/fix/credential-subdomain

 Add ability to match many domains for credentials

Adding more tests to cover different scenario

FIx unrelated failing test

Add ability to match many domains for credentials

Instead of fetching one domain, we use the same method as in site config (to retrieve the matching file) and handle api.example.org, example.org, .org (yes the last one isn’t useful).
If one of these match, we got it and use it.

Merge pull request #3909 from wallabag/fix/html-not-defined

Fix PHP warning

Merge pull request #3908 from wallabag/epub-issue-3642

epub: fix exception when articles have the same title

Merge pull request #3860 from baurmatt/fix/posix_compatibility

Make dev/install/update script posix compatible

Fix PHP warning

Looks like sometimes (usually from import) the `html` key isn’t available.

epub: fix exception when articles have the same title

This commit fixes an exception occuring when exporting as epub several
articles with the same title. The chapter filename is now derived from
title and url.

Fixes #3642

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

Properly source require.sh

Fixes #3571.

Merge pull request #3901 from wallabag/fix-pr-3893

material: fix left padding on non-entry pages introduced by #3893

material: fix left padding on non-entry pages introduced by #3893

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

Merge pull request #3897 from wallabag/2.3.8-dev

Jump to 2.3.8-dev

Jump to 2.3.8-dev

Merge pull request #3895 from wallabag/prepare-2.3.7

Prepare 2.3.7 release

Prepare 2.3.7 release

Merge pull request #3894 from wallabag/issue-3851

material: hide creation date from card actions on specific sizes

material: update prod assets

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

material: add media queries to hide creation date from card actions

Hide the creation date from card actions on specific sizes when there's
not enough space for all parts.

Fixes #3851

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

material: wrap card actions, remove class hiding of creation date

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

Merge pull request #3893 from wallabag/issue-3877

material: fix left padding of content on medium screens

material: fix left padding of content on medium screens

Fixes #3877

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

material: move a media query from cards to dedicated scss file

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

Merge pull request #3885 from mart-e/script-ignore-root

Allow optional --ignore-root-warning

Merge pull request #3886 from wallabag/issue-3602

epub export: fix missing cover image, only for exports of one article

Merge pull request #3887 from wallabag/issue-3711

Add support of expect parameter to change return object when deleting entry

tests: fix broken session clients for rest entry deletion tests

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

EntryRestController: add support of expect parameter to delete action

The expect parameter enables an application to request the whole entry
or only the id when requesting its deletion.

`expects` defaults to `entry` to prevent any API breakage.

Fixes #3711

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

epub export: fix missing cover image, only for exports of one article

Fixes #3602

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

tests: create entry for testDeleteEntry, fix missing id

When using the entity manager to retrieve an already stored entry, the
id disapears from $entry after the first delete call. This leads to
testing a nonexistent endpoint (api/entries/.json) during the second
delete call.

This change now creates an entry specifically for the test.

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

Allow optional --ignore-root-warning

Now both
./install prod --ignore-root-warning
./install --ignore-root-warning prod

will work

$1 was both used for the environment and the root flag

Fixes wallabag/wallabag#3884

Merge pull request #3882 from wallabag/fix/cors

Fix CORS for API

CS

CS

CORS repare for api

Merge pull request #3869 from itsthejb/fix-2factor-email-image

Fix broken 2 factor auth logo image

Fix broken 2 factor auth logo image

Merge pull request #3819 from stan3/pocket-import

Fix read & starred status in Pocket import

Improve checks & add tests

status and favourite are actually strings in the import so use ==

Merge pull request #3861 from wallabag/fix/siteconfig-email

Add dedicated email for site config issue

Add dedicated email for site config issue

Instead of sending an email to the devs, it now creates an issue on GitHub using a zap from zapier.

Merge pull request #3853 from techexo/patch-2

Update composer.json to add php-tidy (ext-tidy)

Update composer.json to add php-tidy (ext-tidy)

Should fix https://github.com/wallabag/wallabag/issues/3844 for milestone https://github.com/wallabag/wallabag/milestone/63.

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