Unit Test for the new ID system

Apply the new ID system accros the whole codebase

Update method to use the new ID system, which replaces linkdate primary keys.

creation and update dates are now DateTime objects.
Since this update is very sensitve (changing the whole database), the datastore will be automatically backed up into the file datastore.<datetime>.php.

Link ID refactoring

    Links now use an incremental unique numeric identifier.
    This ID is persistent and must never change.

    ArrayAccess is used to match the link ID with the array keys (see the comment in LinkDB for more details)

    Key 'created' added, with creation date as a DateTime object. 'updated' is now also a DateTime.

Merge pull request #702 from ArthurHoaro/feed-cdata

Remove new line between content tag and CDATA in ATOM feed

Merge pull request #703 from ArthurHoaro/changelog

Add latest merged changes to the CHANGELOG

Add latest merged changes to the CHANGELOG

Merge pull request #701 from ArthurHoaro/plugins/md-html-doc

Describe markdown HTML rendering and display a warning

Remove new line between content tag and CDATA in ATOM feed

Content not starting directly with CDATA can be misinterpreted by some feed parsers.

Describe markdown HTML rendering and display a warning

Fixes #688

Merge pull request #691 from ArthurHoaro/plugins/no-md-feed

Markdown: fixes feed rendering with nomarkdown tag

Merge pull request #700 from teromene/firefox-social-title

Show page title when sharing via Firefox Social.

Merge pull request #699 from teromene/https-no-social

Disable Firefox Social in the tools section if the page is not loaded using HTTPS.

Show page title when sharing via Firefox Social.

Disable Firefox Social in the tools section if the page is not loaded using HTTPS, as Firefox will deny the request.

Markdown: fixes feed rendering with nomarkdown tag

 * make sure we match exactly `nomarkdown` tag
 * pass the whole link data to stripNoMarkdownTag() to:
   * strip the noMD tag in taglist (array)
   * strip the tag in tags (string)

Fixes #689

tmp

Merge pull request #692 from fpunktk/patch-1

add meta tag to block sending the referrer

Add meta tag to block sending the referrer

Add a meta tag that tells the browser not to send the referrer header to
external sites.

Merge pull request #680 from ArthurHoaro/apache-htaccess

.htaccess files: support Apache 2.4+ syntax

Merge pull request #684 from virtualtam/fix/docker/nginx

Docker: fix & improve nginx+php-fpm configuration

.htaccess files: support Apache 2.4+ syntax

If `mod_version` is enabled, the previous syntax will apply for Apache <2.4.
If not, the new syntax is used by default.

Fixes #676

`mod_version` identifier is `version_module` across all Apache versions. See:

  * https://httpd.apache.org/docs/current/mod/mod_version.html
  * https://httpd.apache.org/docs/2.2/mod/mod_version.html
  * https://serverfault.com/questions/733910/how-do-i-load-mod-version-only-if-it-isnt-built-in-to-apache

Note that version_module comes built-in with Debian (and derivatives) Apache2 packages, see https://wiki.debian.org/Apache/PackagingFor24

Merge pull request #683 from ArthurHoaro/plugins/w3ccompliance

Plugins W3C compliance

Docker: increase maximum file upload size to 10 MiB

Relates to https://github.com/shaarli/Shaarli/issues/681

Changed:
- nginx+php-fpm: set maximum upload size to 10 MiB

See:
- https://nginx.org/en/docs/http/ngx_http_core_module.html#client_max_body_size
- https://secure.php.net/manual/en/ini.core.php#ini.post-max-size
- https://secure.php.net/manual/en/ini.core.php#ini.upload-max-filesize

Signed-off-by: VirtualTam <virtualtam@flibidi.net>

Docker: set favicon location in nginx configuration

Relates to https://github.com/shaarli/Shaarli/issues/681

Fixed:
- nginx: set the favicon location

See http://serverfault.com/a/352861

Signed-off-by: VirtualTam <virtualtam@flibidi.net>

Plugins W3C compliance

Add an alt attribute to images

Merge pull request #677 from yapbreak/master

Piwik Plugin

Add Piwik Plugin

[PullRequest #677] Change after Review

Fix logic, my bad!

Merge pull request #674 from ArthurHoaro/parsedown-composer

Use Composer for Parsedown and fix an issue with links

Fix an issue with links not being reversed in code blocks

Fixes #672

+ Markdown to HTML unit test

Use Composer to import Parsedown library

Reference #613

Merge pull request #673 from virtualtam/cleanup/linkdb

LinkDB: code cleanup

Merge pull request #665 from ArthurHoaro/fix/feed-hashtags

Fix hashtag links in Feeds

LinkDB: explicit method visibility

Relates to https://github.com/shaarli/Shaarli/issues/95

Signed-off-by: VirtualTam <virtualtam@flibidi.net>

LinkDB: update datastore method names

Relates to https://github.com/shaarli/Shaarli/issues/95

Signed-off-by: VirtualTam <virtualtam@flibidi.net>

LinkDB: do not prefix privates with an underscore

Relates to #95

Signed-off-by: VirtualTam <virtualtam@flibidi.net>

Merge pull request #671 from ArthurHoaro/cleanup-code

Minor code cleanup: PHPDoc, spelling, unused variables, etc.

Fix hashtags links in Feeds

Make the hashtag link absolute in feeds to work properly in RSS syndication tools.

Minor code cleanup: PHPDoc, spelling, unused variables, etc.

Merge pull request #670 from virtualtam/travis/composer-cache

Travis: enable Composer cache

Travis: enable Composer cache

See:
- https://docs.travis-ci.com/user/caching/
- https://blog.wyrihaximus.net/2015/07/composer-cache-on-travis/

Signed-off-by: VirtualTam <virtualtam@flibidi.net>

Merge pull request #661 from virtualtam/release/archive

Update release archive generation

Update release archive generation

Relates to https://github.com/shaarli/Shaarli/issues/607
Relates to https://github.com/shaarli/Shaarli/pull/608

Modifications:
- match the arborescence of the archives provided by GitHub
- generate compressed tarballs

Signed-off-by: VirtualTam <virtualtam@flibidi.net>

Merge pull request #664 from virtualtam/changelog

Add CHANGELOG.md

CHANGELOG.md: add an UNPUBLISHED draft for v0.8.1

Relates to #663
See http://keepachangelog.com/en/0.3.0/

Signed-off-by: VirtualTam <virtualtam@flibidi.net>

CHANGELOG.md: improve formatting, fix typos

Relates to #663
See http://keepachangelog.com/en/0.3.0/
See http://sebsauvage.net/wiki/doku.php?id=php:shaarli:history

Modifications:
- keep 2 newlines between each release section
- keep reasonable line lengths (< 90 characters)
- remove extra final dots (only kept when there are several sentences)
- fix typos
- reword/rewrite/curate legacy changelog entries

Signed-off-by: VirtualTam <virtualtam@flibidi.net>

CHANGELOG.md: add legacy Shaarli releases

Relates to https://github.com/shaarli/Shaarli/issues/663
See http://keepachangelog.com/en/0.3.0/
See http://sebsauvage.net/wiki/doku.php?id=php:shaarli:history
See http://sebsauvage.net/wiki/doku.php?id=php:shaarli:history&do=revisions
See http://sebsauvage.net/rhaa/index.php?q=shaarli

These entries correspond to the original (legacy) Shaarli release
notes by Sebsauvage.

Signed-off-by: VirtualTam <virtualtam@flibidi.net>

CHANGELOG.md: add unpublished Shaarli Community releases

Relates to https://github.com/shaarli/Shaarli/issues/663
See http://keepachangelog.com/en/0.3.0/

These entries correspond to the first lightweight tags present on the
Shaarli Community repository. As there are no corresponding release (notes),
the changelog is based on relevant elements from the Git commit log.

Signed-off-by: VirtualTam <virtualtam@flibidi.net>

CHANGELOG.md: add published Shaarli Community releases

Relates to https://github.com/shaarli/Shaarli/issues/663
See http://keepachangelog.com/en/0.3.0/

This file mirrors the content of the Shaarli release notes available at
https://github.com/shaarli/Shaarli/releases

Signed-off-by: VirtualTam <virtualtam@flibidi.net>

Merge pull request #651 from ArthurHoaro/plugin-isso2

Isso comments plugin

Merge pull request #662 from virtualtam/fix/feed/self-link

Fix: return the proper value for the "self" feed attribute

Merge pull request #660 from virtualtam/fix/template/loop

Cleanup: explicitely loop over PHP variables in templates

Isso comments plugin

Use Isso client to let visitors comments on permalinks

Merge pull request #659 from ArthurHoaro/plugin-errors

New init function for plugins, supports errors reporting

Fix: return the proper value for the "self" feed attribute

Fixes https://github.com/shaarli/Shaarli/issues/629
Closes https://github.com/shaarli/Shaarli/pull/630

Note: you might need to empty the "pagecache" directory for the
fix to be taken into account

Signed-off-by: VirtualTam <virtualtam@flibidi.net>

Cleanup: explicitely loop over PHP variables in templates

Relates to https://github.com/shaarli/Shaarli/issues/613

Before: {loop="someVariable"}
After:  {loop="$someVariable"}

Signed-off-by: VirtualTam <virtualtam@flibidi.net>

Merge pull request #658 from ArthurHoaro/hotfix/plugin-desc-bug

Bugfix: display plugin parameter description only if it exists

New init function for plugins, supports errors reporting

All plugins can optionally add an init function named `pluginname_init()` which is called when the plugin is loaded.

This function is aware of the config, and can return initialization errors, which are displayed in the header template.

Note that the previous error system hack no longer work.

Bugfix: display plugin parameter description only if it exists

Merge pull request #654 from teromene/archive-org-no-internal

Archive.org plugin: do not propose archival of private notes

Fixes #637

Archive.org plugin: do not propose archival of private notes
Fixes #637

Merge pull request #622 from ArthurHoaro/update-date

Save link update dates and render it in templates and feeds

Merge pull request #623 from ArthurHoaro/security/reverse-proxy-ban

Add trusted IPs in config and try to ban forwarded IP on failed login

Merge pull request #656 from ArthurHoaro/v0.8.0

Bump version to v0.8.0

Bump version to v0.8.0

Signed-off-by: ArthurHoaro <arthur@hoa.ro>

Merge pull request #649 from krypty/dockerfile-issue

Add a missing backslash in stable Dockerfile version

Add a missing backslash in stable Dockerfile version

Merge pull request #646 from virtualtam/composer/check-autoload

composer: display an error message if the autoload script is missing

composer: display an error message if the autoload script is missing

Closes https://github.com/shaarli/Shaarli/issues/645
Relates to https://github.com/shaarli/Shaarli/issues/607

Signed-off-by: VirtualTam <virtualtam@flibidi.net>

README: fix link address typo

Fixes https://github.com/shaarli/Shaarli/issues/642

Signed-off-by: VirtualTam <virtualtam@flibidi.net>

update link to install instuctions in README

Merge pull request #638 from virtualtam/docker/stable/composer

Docker: update dependencies, introduce Composer (stable branch)

Merge pull request #633 from virtualtam/docker/composer

Docker: update dependencies, introduce Composer

Docker: update dependencies, introduce Composer

Relates to https://github.com/shaarli/Shaarli/issues/607
Relates to https://github.com/shaarli/Shaarli/pull/612
Relates to https://github.com/shaarli/Shaarli/pull/624
Relates to https://github.com/shaarli/Shaarli/pull/633

See https://github.com/shaarli/Shaarli/wiki/Server-requirements

Modifications:
- [prod][stable] refactor Dockerfile
- [prod][stable] set $TERM=dumb to avoid debconf-related issues
- [prod][stable] install ca-certificates
- [prod][stable] cleanup APT cache after installing packages
- [prod][stable] use Composer to resolve PHP dependencies

Signed-off-by: VirtualTam <virtualtam@flibidi.net>

Docker: update dependencies, introduce Composer

Relates to https://github.com/shaarli/Shaarli/issues/607
Relates to https://github.com/shaarli/Shaarli/pull/612
Relates to https://github.com/shaarli/Shaarli/pull/624

See https://github.com/shaarli/Shaarli/wiki/Server-requirements

Modifications:
- [all][env] set $TERM=dumb to avoid debconf-related issues
- [all][pkg] install ca-certificates
- [all][pkg] cleanup APT cache after installing packages
- [dev] refactor Dockerfile
- [prod][master] refactor Dockerfile
- [prod][master][php] use Composer to resolve PHP dependencies

Signed-off-by: VirtualTam <virtualtam@flibidi.net>

Merge pull request #619 from ArthurHoaro/plugins/param-desc

Add a description to plugin parameters

Merge pull request #636 from virtualtam/fix/final-newline

Fix: add missing final newlines, untabify text

Fix: add missing final newlines, untabify text

Signed-off-by: VirtualTam <virtualtam@flibidi.net>

Merge pull request #635 from virtualtam/fix/import/ie

Fix: ensure Internet Explorer bookmark dumps can be imported

Fix: ensure Internet Explorer bookmark dumps can be imported

Relates to https://github.com/shaarli/Shaarli/issues/607

Modifications:
- [application][tests] NetscapeBookmarkUtils: more permissive doctype detection

The IE bookmark exports contain extra escape sequences, which can be observed
by binary comparison of the reference input data used in tests:

   $ cmp -b -l -n 8 netscape_basic.htm internet_explorer_encoding.htm

   1  74 <    357 M-o
   2  41 !    273 M-;
   3 104 D    277 M-?
   4 117 O     74 <
   5 103 C     41 !
   6 124 T    104 D
   7 131 Y    117 O
   8 120 P    103 C

Signed-off-by: VirtualTam <virtualtam@flibidi.net>

Merge pull request #634 from virtualtam/composer/gitattributes

Fix: keep composer.json in release archives

Fix: keep composer.json in release archives

Relates to https://github.com/shaarli/Shaarli/issues/607

Signed-off-by: VirtualTam <virtualtam@flibidi.net>

Merge pull request #612 from virtualtam/refactor/bookmark-parser

Refactor bookmark import using a generic Netscape parser

Refactor bookmark import using a generic Netscape parser

Relates to #607
Relates to #608
Relates to #493 (abandoned)

Additions:
- use Composer's autoload to load 3rd-party dependencies under vendor/

Modifications:
- [import] replace the current parser with a generic, stable parser
  - move code to application/NetscapeBookmarkUtils
  - improve status report after parsing
- [router] use the same endpoint for both bookmark upload and import dialog
- [template] update bookmark import options
  - allow adding tags to all imported links
  - allow selecting the visibility (privacy) of imported links
- [tests] ensure bookmarks are properly parsed and imported in the LinkDB
  - reuse reference input from the parser's test data

See:
- https://github.com/shaarli/netscape-bookmark-parser
- https://getcomposer.org/doc/01-basic-usage.md#autoloading

Signed-off-by: VirtualTam <virtualtam@flibidi.net>

Merge pull request #632 from virtualtam/composer/shaarli/netscape-bookmark-parser

Composer: reference shaarli/netscape-bookmark-parser from Packagist

Composer: reference shaarli/netscape-bookmark-parser from Packagist

Relates to https://github.com/shaarli/Shaarli/pull/607
Relates to https://github.com/shaarli/Shaarli/pull/612
Relates to https://github.com/shaarli/netscape-bookmark-parser/issues/15

Modification:
- reference the "shaarli" vendor repository on Packagist instead of
  overriding the upstream package with an SCM repository

See https://packagist.org/packages/shaarli/netscape-bookmark-parser

Signed-off-by: VirtualTam <virtualtam@flibidi.net>

Merge pull request #624 from julienCXX/pr-curl-http-fetch

Added (and set as default) a cURL-based method for fetching HTTP content

Set cURL as default in HTTP fetching, a fallback method and consistency fixup between both methods

Merge pull request #628 from ArthurHoaro/hotfix/default-private

Bugfix: wrong key used to get default private links setting

Bugfix: wrong key used to get default private links setting

Merge pull request #627 from ArthurHoaro/feature/translate-init

Initialize a translation function

Initialize a translation function

It matches the API of ngettext().

Merge pull request #625 from ArthurHoaro/token

Generate a token for every pages

Merge pull request #626 from ArthurHoaro/hotfix/changepassword

Bugfix: enable change password if open shaarli is disabled

Bugfix: enable change password if open shaarli is disabled

Generate a token for every pages

Add trusted IPs in config and try to ban forwarded IP on failed login

  * Add a new settings (which needs to be manually set): `security.trusted_proxies`
  * On login failure, if the `REMOTE_ADDR` is in the trusted proxies, try to retrieve the forwarded IP in headers.
  * If found, the client address is added in ipbans, else we do nothing.

Fixes #409

Set updated date for items in feeds

RSS doesn't support updated date for items, so we use the ATOM extension.
Updated dates also bump the global update

Save the update date in LinkDB and pass it to linklist templates

It can be used as a timestamp by templates under the key 'updated_timestamp'.

Merge pull request #621 from ArthurHoaro/hotfix/update-escape-config

Fix update method escapeUnescapedConfig

Merge pull request #618 from ArthurHoaro/tagclean

Better whitespace handling in tags