CS

Replace continue; with break; to avoid PHP 7.3 warnings

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

Merge pull request #3879 from wallabag/fix/url-default-protocol

Force default_protocol to generate an url input

Force default_protocol to generate an url input

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

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 #3798 from wallabag/update-two-factor-bundle

Enable OTP 2FA

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 ==

Hash backup codes in the database using `password_hash`

Revert PG on Travis about drop/create the database

CS

Change the way to enable 2FA

And add a step to validate a generated code from the OTP app

Remove secret from admin

Update translation

Fix PG & Travis drop/create database

Add SQLite & PG migration

Also remove the forced `server_version` from dbal config to avoid an
hard overriding across all database.

Add backup codes

Better translations

Replace “Google Authenticator” by “Google Authenticator, Authy or FreeOTP” in all text.

Translate how to use the code / qr code.

Cleanup

Fix tests

Fix test for custom version of the tidy extension

Enable OTP 2FA

- Update SchebTwoFactorBundle to version 3
- Enable Google 2fa on the bundle
- Disallow ability to use both email and google as 2fa
- Update Ocramius Proxy Manager to handle typed function & attributes (from PHP 7)
- use `$this->addFlash` shortcut instead of `$this->get('session')->getFlashBag()->add`
- update admin to be able to create/reset the 2fa

Create a dedicated tab to reset data

Merge pull request #3627 from craig0990/feature/add-search-api-endpoint

Add a basic Search REST endpoint

Merge pull request #3864 from wallabag/fix/database-init-version

Removed hardcoded database version

Avoid creating PG database on Travis

And Travis build on PHP 7.3 should not fail.

Fix third argument to Route

Factorize sendResponse between Api controllers

And run newer cs fixer

[wallabag/wallabag#2611] Fix PHPCS lint errors

Fixes [wallabag/wallabag#2611] Add a basic Search REST endpoint

- Adds a new `search` key to `src/Wallabag/ApiBundle/Resources/config/routing_rest.yml`
- Reuses the `getBuilderForSearchByUser` method from the EntryRepository
- Supports, `term`, `page`, and `perPage` query parameters
- Some very basic tests

Removed hardcoded database version

Merge pull request #3526 from wallabag/add-random-article

Add random feature

Update icon to `casino`

Handle no random result found

Fix status from Instapaper

That status was used as `http_status` in ContentProxy->stockEntry

Move icon into the top menu bar

Change the way to select a random entry:
- select all ids from the given user (with filters)
- choose randomly one in php
- find that entry

Redirect to the current view instead of homepage

Better icon

Better random function

Use only one method to randomize

Instead of one per type, one for all is ok.

Fix tests

Added random feature

Merge pull request #3858 from wallabag/bump-ocramius

Bump ocramius/proxy-manager

CS

Merge pull request #3859 from wallabag/phpstan

Enable PHPStan

Enable PHPStan

- Fix error for level 0 & 1 (level 7 has 699 errors...)
- Add `updated_at` to site_credential (so the `timestamps()` method applies correctly)

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.

Bump ocramius/proxy-manager

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

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 #3845 from wallabag/feature/api-info-endpoint

Add a new endpoint to retrieve information from the wallabag instance

Add a new endpoint to retrieve information from the wallabag instance

Useful for api client which required some information.
We might add more inside them in the future.

The endpoint /api/version should be avoided now as it contains not so much information rather the version.

CS

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