Replaced PHP_EOL  to "\n"

i was getting error

```
An error occurred while parsing JSON configuration file (data/config.json.php): error code #4
➜ Syntax error
Please check your JSON syntax (without PHP comment tags) using a JSON lint tool such as jsonlint.com.
```
after debug i found
```php
        $data = str_replace(self::getPhpHeaders(), '', $data);
        $data = str_replace(self::getPhpSuffix(), '', $data);
```
doesn't removing php header and php suffix

cause of this issue was PHP_EOL represents the endline character for the current system. if my  ```config.json.php```  was encoded with unix ( LF ) and php running on windows windows encoding ( CR LF ) is not same as unix encoding ( LF ) so ```str_replace``` doesn't replace strin  then it causes issue.

Merge pull request #1601 from ArthurHoaro/feature/psr3

Merge pull request #1605 from ArthurHoaro/fix/nginx-doc-rule

Fix: nginx - add rule to disable url-rewriting for the docs

Fix: nginx - add rule to disable url-rewriting for the docs

Related to #1603

Merge pull request #1602 from ArthurHoaro/fix/root-exceptions

Dislay an error if an exception occurs in the error handler

Dislay an error if an exception occurs in the error handler

Related to #1598

Use PSR-3 logger for login attempts

Fixes #1122

Composer: explicitly import katzgrau/klogger (already included in netscape-bookmark-parser)

Merge pull request #1584 from ArthurHoaro/feature/async-thumbnail-retrieval

Asynchronous retrieval of bookmark's thumbnails

Asynchronous retrieval of bookmark's thumbnails

This feature is based general.enable_async_metadata setting and works with existing metadata.js file.
The script is compatible with any template:
   - the thumbnail div bloc must have  attribute
   - the bookmark bloc must have  attribute with the bookmark ID as value

Fixes #1564

Merge pull request #1567 from ArthurHoaro/feature/async-title-retrieval

Merge pull request #1600 from yudete/master

Update translations (Japanese)

Update translations (Japanese)

Merge pull request #1588 from ArthurHoaro/feature/search-highlight

add search highlight unit tests

Feature: highlight fulltext search results

How it works:

  1. when a fulltext search is made, Shaarli looks for the first
occurence position of every term matching the search. No change here,
but we store these positions in an array, in Bookmark's additionalContent.
  2. when formatting bookmarks (through BookmarkFormatter
implementation):
    1. first we insert specific tokens at every search result positions
    2. we format the content (escape HTML, apply markdown, etc.)
    3. as a last step, we replace our token with displayable span
elements

Cons: this tightens coupling between search filters and formatters
Pros: it was absolutely necessary not to perform the
search twice. this solution has close to no impact on performances.

Fixes #205

Merge pull request #1596 from ArthurHoaro/feature/better-rename-tag

Improve Manage tags page

Improve Manage tags page

Fixes #1125

Merge pull request #1593 from ArthurHoaro/fix/no-url-rewriting

Doc: typo

Inject ROOT_PATH in plugin instead of regenerating it everywhere

Support using Shaarli without URL rewriting

   - Shaarli can be fully used by prefixing any URL with /index.php/
   - {$base_path} used in templates already works with this configuration
   - Assets path (outside of theme's assets) must be prefixed with {$root_url}/
   - Documentation section in « Server configuration »

Fixes #1590

Merge pull request #1592 from ArthurHoaro/fix/strict-type-daily

Strict types: fix an issue in daily where the date could be an int

Strict types: fix an issue in daily where the date could be an int

Improve metadata retrieval (performances and accuracy)

  - Use dedicated function to download headers to avoid apply multiple regexps on headers
  - Also try to extract title from meta tags

Add a setting to retrieve bookmark metadata asynchrounously

  - There is a new standalone script (metadata.js) which requests
    a new controller to get bookmark metadata and fill the form async
  - This feature is enabled with the new setting: general.enable_async_metadata
    (enabled by default)
  - general.retrieve_description is now enabled by default
  - A small rotating loader animation has a been added to bookmark inputs
    when metadata is being retrieved (default template)
  - Custom JS htmlentities has been removed and  mathiasbynens/he
    library is used instead

Fixes #1563

Merge pull request #1591 from ArthurHoaro/doc/server-conf-php-v

Doc: add PHP 7.4 and 8.0 as supported version

Doc: add PHP 7.4 and 8.0 as supported version

Merge pull request #1583 from ArthurHoaro/feature/bookmark-strict-types

Add strict types for bookmarks management

Add strict types for bookmarks management

Parameters typing and using strict types overall increase the codebase
quality by enforcing the a given parameter will have the expected type.

It also removes the need to unnecessary unit tests checking methods
behavior with invalid input.

Merge pull request #1570 from ArthurHoaro/feature/datastore-mutex

Add mutex on datastore I/O operations

Add mutex on datastore I/O operations

To make sure that there is no concurrent operation on the datastore file.

Fixes #1132

Merge pull request #1540 from ArthurHoaro/fix/metadata-regexes

Improve regex to extract HTML metadata (title, description, etc.)

Merge pull request #1525 from ArthurHoaro/feature/rest-api-bookmark-dates

REST API: allow override of creation and update dates

Merge pull request #1521 from ArthurHoaro/feature/markdown-extra

Add Markdown Extra formatter

Add Markdown Extra formatter

Library: [Parsedown Extra](https://github.com/erusev/parsedown-extra)

Also sort dependencies alphabetically.

Fixes #1169

Update badge versions

Merge pull request #1589 from ArthurHoaro/changelog/v0.12.0

CHANGELOG and AUTHORS for v0.12.0

CHANGELOG and AUTHORS for v0.12.0

Merge pull request #1586 from ArthurHoaro/changelog/v0.12.0-beta-2

CHANGELOG and AUTHORS for v0.12.0-beta-2

CHANGELOG and AUTHORS for v0.12.0-beta-2

Merge pull request #1585 from ArthurHoaro/fix/xss-and-tag-search

Security: fix multiple XSS vulnerabilities + fix search tags with special chars

Security: fix multiple XSS vulnerabilities + fix search tags with special chars

XSS vulnerabilities fixed in editlink, linklist, tag.cloud and tag.list.

Also fixed tag search with special characters: urlencode function needs to be applied on raw data, before espaping, otherwise the rendered URL is wrong.

Merge pull request #1579 from sprak3000/issue-1437-tag-sort-buttons-ui

Fix #1437 - Make tag cloud/list views buttons more obvious

Merge pull request #1581 from nodiscc/compose-traefik-version

docker-compose.yml: pin traefik image to 1.7-alpine

docker-compose.yml: pin traefik image to 1.7-alpine
- fixes https://github.com/shaarli/Shaarli/issues/1493
- https://hub.docker.com/_/traefik/

Merge pull request #1575 from ArthurHoaro/feature/php8

Merge pull request #1574 from stoeps13/hosting-fix

 Issue #1437 - Make tag cloud/list views buttons more obvious

 This work alters the markup and CSS for the tag sort UI to match the button feel filters and links per page uses.

Merge pull request #1577 from ArthurHoaro/fix/edit-zero

Fix a bug preventing to edit bookmark with ID #0

Revert unrelated changes and add unit tests

Fix a bug preventing to edit bookmark with ID #0

Fix identation

Change  to ->container->environment

Merge pull request #1576 from ArthurHoaro/release/v0.12.0-beta-1/changelog

Update CHANGELOG and AUTHOR

Update CHANGELOG and AUTHOR

Merge pull request #1569 from ArthurHoaro/fix/bad-encoding

Fix warning if the encoding retrieved from external headers is invalid

Merge pull request #1566 from nodiscc/makefile-composer-install

doc/Makefile: remove references to composer update

Fix warning if the encoding retrieved from external headers is invalid

Also fixed the regex to support this failing header: charset="utf-8"\r\n"

Use assertRegExp polyfill instead of regexMatches

Ignore PHP deps when removing phpunit in PHP 8.0 environment

Update composer dependencies from PHP 7.1

Fix missing @expectedException convertion

Compatibility with PHP 8

Compatibility with PHPUnit 9

Run CI against PHP 8.0

Workaround for hoster (ionos)

The hoster writes the environment variable with bearer token to
REDIRECT_HTTP_AUTHORIZATION and needs to provide RewriteBase / to
.htaccess

Merge pull request #1572 from ArthurHoaro/feature/php8

Convert legacy PHPUnit @expected* to new ->expect*

Converted automatically using https://github.com/ArthurHoaro/convert-legacy-phpunit-expect

Comply with PHPUnit V8: setup/teardown functions must return void

tmp

Travis CI: run composer update instead of install

And ignore PHP 7.1 platform requirement, in order to get matching version of PHPUnit

Merge pull request #1568 from ArthurHoaro/fix/vintage-linkdate

Fix undefined linkdate variable in vintage theme

Fix undefined linkdate variable in vintage theme

Fixes #1371

Merge pull request #1560 from ArthurHoaro/fix/redirect-wrong-path

Fix invalid redirection using the path of an external domain

Merge pull request #1558 from ArthurHoaro/fix/plugins-base-path

Fix plugin base path in core plugins

Merge pull request #1565 from nodiscc/rm-makefile-summary

Makefile: remove static_analysis_summary from all: target

doc/Makefile: remove references to composer update
- add make composer_dependencies_dev Makefile target and use this instead
- fix composer initial installation procedure
- fix php-xdebug install instructions

Makefile: remove static_analysis_summary from all: target
static_analysis_summary was removed in 37c9c6b#diff-b67911656ef5d18c4ae36cb6741b7965 but not from the all: target dependencies. Therefore running make all always fails.
fixes https://github.com/shaarli/Shaarli/issues/1459

Merge pull request #1561 from ArthurHoaro/feature/front-deps-upgrade

Fix ESLint after dependency upgrade

Bump NodeJS version on travis build

Upgrade front end dependencies

Mostly in order to get rid of deprecated deps, and upgrade vulnerable ones.

  - Upgrade webpack from 3.x to 4.x
  - Moved babel package to main repo
  - Replaced deprecated extract-text-webpack-plugin with extract-text-webpack-plugin
  - Replaced deprecated babel-minify-webpack-plugin with terser-webpack-plugin
  - Replaced deprecated node-sass with (dart) sass package
  - Replaced deprecated sass-lint with stylelint (the rules might be a bit different

Related to #1531: trivy doesn't raise any more issue

Fix invalid redirection using the path of an external domain

Fixes #1554

Merge pull request #1559 from ArthurHoaro/fix/edit-redirect

Fix redirection to referer after editing a link

Merge pull request #1539 from ArthurHoaro/feature/manual-root-url

Fix redirection to referer after editing a link

Fixes #1545

Fix plugin base path in core plugins

Also fix note check in archiveorg plugin, and regression on vintage template.
Documentation regarding relative path has been added.

Fixes #1548

Merge pull request #1556 from kcaran/apache_methods

Added PATCH to the allowed Apache request methods.

Merge pull request #1553 from ArthurHoaro/fix/404-page

Properly handle 404 errors

Merge pull request #1551 from ArthurHoaro/fix/plugin-save-metadata

Plugins: do not save metadata along plugin parameters

Fix subfolder configuration in unit tests

Add manual configuration for root URL

This new setting under 'general.root_url' allows to override automatic discovery of Shaarli instance's URL.

Fixes #1339

Merge pull request #1389 from shaarli/doc-rework-setup

doc: rework installation/setup guides, general refactoring

doc: Docker minor improvements

Add 2 plugins to the 3rd party plugin list

Besides adding 2 new plugins, also reordered the list by ABC and fixed some discrepancies in the details to restore balance in the force

doc: nginx config: document ipv4 and ipv6 listen directives

doc: apache config: remove useless documentroot directive in HTTP-only virtualhost (only used for redirects)