Cleanup

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

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 #3574 from shulard/feature/rename-tags

Allow to rename tags from the web interface.

Merge pull request #3216 from wallabag/change-locale-register

Added possibility to change locale from login/register pages

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

Ensure language is valid

- Do not override locale if user has choosen a locale from the login screen.
- Add some tests about locale url

Added possibility to change locale from login/register pages

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.

Merge pull request #3596 from glego/add-secure-email

Add secure email

Add secure email

Merge pull request #3731 from wallabag/symfony3.4

Jump to Symfony 3.4

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)

Jump to Symfony 3.4

Thanks to the BC compatibility, almost nothing have to be changed.
All changes are related to new bundle version of:
- SensioFrameworkExtraBundle
- DoctrineFixturesBundle

Update assets

Add tests about the tag renaming process.

Add specific styles for the card tag form element.

Also add a `.hidden` class in the baggy theme to have consistency with material.

Add translations about latest Tag changes.

Add new translations in each language file.

Update tag list template to allow renaming.

* Add a form on each tag to handle rename action.
* Add JavaScript to handle action on the corresponding page inside the global index.js file.
* Add support for the 2 active themes : material / baggy

The form solution is cleaner than an Ajax one because it let the browser validate input data and make the POST easier without the need to handle JSON response.

Add RenameForm as tag list view parameters.

This will help handling the CSRF protection token and use symfony HTML generation layer.
Also a FormView instance is generated for each tag because we need to render a form for each tag and FormView are not reusable.

Create a new Tag action to rename tags.

The current tag is removed from all the current logged user entries. Then the new one is created and attached.

Rename Tag : Add a new FormType

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 #3630 from sviande/archived_at

Entry: add archived_at property and updateArchived method

Sort archive page by archived at

Code Style

Entry: add sort parameter archived

Entry: add archived_at property and updateArchived method

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

Merge pull request #3726 from wallabag/fix-tests

Liberation goes https

Liberation goes https

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

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 remote-tracking branch 'origin/master' into 2.4

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

Merge pull request #3705 from wallabag/update-release

Update release script

Makefile fixes for non GNU systems

* Move `Makefile` to `GNUmakefile`, which GNU `make` picks first
* Add `Makefile` so other `make`s forward to `gmake`
* Set the `SHELL` variable and let `make` handle the shell

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