Bugfix: do not store plugin errors in data.php

Before this, calling writeConfig() would have write error messages in data.php, because it uses 'plugins' array which is used for plugin configuration.

Causing the message error appear everytime.

Merge pull request #383 from ArthurHoaro/master

Bump version to v0.6.0

Bump version to v0.6.0

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

Merge pull request #374 from virtualtam/cleanup

cleanup: remove the executable bit from source scripts

cleanup: remove the executable bit from source scripts

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

Merge pull request #373 from virtualtam/index/format-globals-comments

index.php: group globals by theme, format comments

index.php: group globals by theme, format comments

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

Merge pull request #275 from shaarli/plugin-proposition

Plugin proposition

Handle errors raised by plugins in template. fixes #370

Merge pull request #286 from ArthurHoaro/plugin-demo_plugin

PLUGIN demo_plugin

PLUGIN demo_plugin

This plugin try to cover Shaarli's plugin API entirely.
Can be used by plugin developper to make their own.

Merge pull request #281 from ArthurHoaro/plugin-wallabag

PLUGIN wallabag

Merge pull request #280 from ArthurHoaro/plugin-readityourself

PLUGIN readityourself

Merge pull request #279 from ArthurHoaro/plugin-addlink_toolbar

PLUGIN: addlink_toolbar

unit tests for the wallabag plugin
+ removed exit error if the config is not found
+ coding style

unit tests for readityourself plugin + remove hard error

unit test for addlink_toolbar + coding style

PLUGIN: addlink_toolbar

Add a field in linklist page to paste a new link.

Displayed in fields toolbar.

PLUGIN readityourself

Add an icon for each link (linklist) for ReadItYourself

Merge pull request #284 from ArthurHoaro/plugin-playvideos

PLUGIN playvideos

Merge pull request #285 from ArthurHoaro/plugin-qrcode

PLUGIN QRCode

Merge pull request #283 from ArthurHoaro/plugin-archiveorg

PLUGIN archiveorg

Merge pull request #368 from ArthurHoaro/returnurl-again

Fixes #356 -  adding a link should return added link's hash

PLUGIN wallabag

Add a Wallabag icon in linklist for each link.

Add unit test for archiveorg plugin
+ coding style

PLUGIN archiveorg

Display an archive.org icon in linklist, foreach links.

Add unit tests for Playvideos plugin
+ coding style

PLUGIN playvideos

Display a button in buttons toolbar which allows to play all videos found.

Add unit tests for the QRCode plugin
+ coding style

PLUGIN QRCode

Add an icon in linklist to display links QRCode

Add plugins folder to test analysis

Unit tests for Router and PluginManager.

Template upgrade to handle plugin zones

Add a bunch of plugin placeholders in templates

Plugins TODO.md

Plugin system - CORE

see shaarli/Shaarli#275

Fixes #356
 * adding a link should return added link's hash
* allow redirection relative urls in generateLocation

Bump version to 0.5.4

Fixes:
 - PHP session IDs: handle hash algorithms and bits per char representations

Minor changes:
 - HTTPS: support being served behing an SSL-enabled proxy
 - HTTP/Server utilities: refactor & add test coverage

Project & documentation:
 - improve/rewrite `README.md`
 - update contributor list
 - update `index.php` header

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

Update project information: contributors, `index.php` header

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

Merge pull request #346 from virtualtam/refactor/http-url-utils

HTTP: move server URL functions to `HttpUtils.php`

HTTP: move server URL functions to `HttpUtils.php`

Relates to #333

Modifications:
 - refactor server URL utility functions
 - do not access global `$_SERVER` variables
 - add test coverage
 - improve readability
 - apply coding conventions

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

SSL detection: add support for `X-Forwarded-Proto`

Duplicates #332

See:
 - RFC 7239 - Forwarded HTTP Extension
   http://www.ietf.org/rfc/rfc7239.txt
 - RFC 6238 - Deprecating the "X-" Prefix and Similar Constructs in Application Protocols
   http://www.ietf.org/rfc/rfc6648.txt
 - StackOverflow - Custom HTTP headers: naming conventions
   http://stackoverflow.com/a/3561399

Merge pull request #337 from doc75/doublon_url

#325 small enhancement to fix the GetLinkFromUrl method

Url: introduce global helper functions for cleanup and scheme detection

Relates to #314 & #326

Additions:
 - add global `cleanup_url()` and `get_url_scheme()` functions

Modifications:
 - replace `Url` usage in `index.php` by calls to global functions
 - fix `Url` tests not being run: PHPUnit expects a single test class per file
   - move classes to separate files

Merge pull request #334 from virtualtam/refactor/http-utils

HTTP: move utils to a proper file, add tests

HTTP: move utils to a proper file, add tests

Relates to #333

Modifications:
 - move HTTP utils to 'application/HttpUtils.php'
 - simplify logic
   - replace 'http_parse_headers_shaarli' by built-in 'get_headers()'
   - remove superfluous '$status' parameter (provided by the HTTP headers)
 - apply coding conventions
 - add test coverage (unitary only)

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

Merge pull request #338 from virtualtam/fix/unique-uniqid

Session ID: extend the regex to match possible hash representations

Session ID: extend the regex to match possible hash representations

Improves #306
Relates to #335 & #336
Duplicated by #339

Issues:
 - PHP regenerates the session ID if it is not compliant
 - the regex checking the session ID does not cover all cases
   - different algorithms: md5, sha1, sha256, etc.
   - bit representations: 4, 5, 6

Fix:
 - `index.php`:
   - remove `uniqid()` usage
   - call `session_regenerate_id()` if an invalid cookie is detected
 - regex: support all possible characters - '[a-zA-Z,-]{2,128}'
 - tests: add coverage for all algorithms & bit representations

See:
 - http://php.net/manual/en/session.configuration.php#ini.session.hash-function
 - https://secure.php.net/manual/en/session.configuration.php#ini.session.hash-bits-per-character
 - http://php.net/manual/en/function.session-id.php
 - http://php.net/manual/en/function.session-regenerate-id.php
 - http://php.net/manual/en/function.hash-algos.php

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

Merge pull request #344 from virtualtam/copying

COPYING: update contributor list

COPYING: update contributor list

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

Update README.md

Merge pull request #343 from virtualtam/readme

Rewrite README.md

Rewrite README.md

Modifications:
 - group content in sections
 - homogenize formatting
 - replace installation instructions by links to the corresponding wiki pages
 - update badges
   - use http://shields.io/ to generate SVGs with custom labels
   - master branch: update Travis label
   - stable branch: add Travis status
   - GitHub release: display the latest released version

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

Merge pull request #340 from virtualtam/doc/update

Doc: sync from Wiki, generate HTML

Doc: sync from Wiki, generate HTML

Additions:
 - Installation/Download: how to get Shaarli
 - Community software: ShaarliOS app

Modifications:
 - Installation/Server requirements: PHP 5.4 EOL, PHP 7 announcements
 - Installation/Server configuration: improve Nginx security
 - Troubleshooting: PHP sessions on `free.fr`

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

Bump version to v0.5.3

Fixes a bug that could prevent user to login.

Merge pull request #336 from ArthurHoaro/login-hotfix

Allow uppercase letters in PHP sessionid format

Allow uppercase letters in PHP sessionid format

Fixes shaarli/Shaarli#335 - Wrong login/password since v0.5.2

Regression introduced in 06b6660a7e8891c6e1c47815cf50ee5b2ef5f270

Bump version to 0.5.2

Minor changes
 - fix Full Path Disclosure upon cookie forgery
 - fix regression preventing to load LinkDB info when adding an existing link
 - also extract HTTPS page metadata (title)
 - add PHP 7 to Travis platforms

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

Merge pull request #326 from ArthurHoaro/bug-url

Fixes #325 - Shaarli does not recognize saved links

Add HTTPS support for title extracting feature

Fixes #325 - Shaarli does not recognize saved links

PHP doesn't seem to autoconvert objects to strings when they're use as array indexes.

Fixes regression introduced in d9d776af19fd0a191f82525991dafbb56e1bcfcb

Merge pull request #306 from ArthurHoaro/fpd

Avoid Full Path Disclosure error on session error.

Merge pull request #327 from virtualtam/travis/php7

travis: add PHP 7 to the tested environments

Avoid Full Path Disclosure error on session error.

  * Add a function to validate session ID.
  * Generate a new session ID if an invalid token is passed.

travis: add PHP 7 to the tested environments

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

Bump version to 0.5.1

Minor changes
 - fix 404 after editing a link while being logged out
 - update local documentation
 - improve timezone detection at installation
 - improve feed cache handling
 - improve URL cleanup for new links
 - add a link to the shaarli/shaarli DockerHub repository

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

Doc: sync from Wiki, generate HTML

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

Merge pull request #314 from shaarli/clean-utm_term

clean utm_term url parameter

README: add DockerHub badge

See [docker-shaarli](https://github.com/shaarli/docker-shaarli) for Dockerfiles and documentation

Links: refactor & improve URL cleanup

Relates to #141
Relates to #133

Modifications
 - move URL cleanup to `application/Url.php`
 - rework the cleanup function
   - fragments: `#stuff`
   - GET parameters: `?var1=val1&var2=val2`
 - add documentation (APIs the params belong to)
 - add test coverage

Reference
 - http://php.net/parse_url
 - http://php.net/manual/en/language.oop5.magic.php#language.oop5.magic.tostring

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

Merge pull request #309 from virtualtam/refactor/PageCache

CachedPage: move to a proper file, add tests

Cache: simplify cached content cleanup, improve tests

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

CachedPage: move to a proper file, add tests

Modifications
 - rename `pageCache` to `CachedPage`
 - move utilities to `Cache`
 - do not access globals
 - apply coding rules
 - update LinkDB and test code
 - add test coverage

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

Merge pull request #301 from ArthurHoaro/edit-link-redirect

Fixes #299: prevent 404 on '?edit_link' while logged out

Fixes #299: prevent 404 on '?edit_link' while logged out

 - add a use case for edit_link in logged out part.
 - *really* prevent loops on login screen.

Merge pull request #313 from virtualtam/install/timezone

Installation: default to the server's timezone

Installation: default to the server's timezone

Modifications
 - attempt to use the server's timezone
 - if none is set, use UTC
 - TimeZone: apply coding conventions
   - variable naming
   - no closing PHP tag

Relates to #274

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

Merge pull request #294 from virtualtam/doc/update

Doc: sync from Wiki, generate HTML

Doc: sync from Wiki, generate HTML

Closes #291
Fixes #227

Modifications
 - HTML content: match the new Wiki structure
 - Makefile
   - generate a custom HTML sidebar
   - include the sidebar on all pages
   - infer and prepend page titles
   - handle relative links
   - add title metadata, e.g. Shaarli - <Page Name>

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

Merge pull request #276 from virtualtam/tools/phpcs

Add a generic rule to run PHPCS against different coding standards

Merge pull request #303 from virtualtam/v0.5.0

Bump version to 0.5.0

Bump version to 0.5.0

Major changes
 - fix locale handling
 - fix note URLs
 - fix page redirections
 - fix daily RSS browsing
 - fix title display
 - fix links not being hidden when `HIDE_PUBLIC_LINKS` is set
 - restore compatibility with PHP 5.3
 - remove duplicate tags in links
 - remove annoying URL patterns
 - add Firefox Social API
 - Search/Filter by tag fieds can now be accessed quickly with the `Tab` key
 - update documentation
 - start code refactoring
   - move all settings to `data/config.php`
   - refactor Config, LinkDB, TimeZone, Utils
   - add unit test coverage
   - add Travis integration

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

Add Requirements section in README (link to wiki).

Fixes #297

Merge pull request #295 from Knah-Tsaeb/patch-1

[fix]  #293 - Black thumbnails on picture wall after upgrade

Merge pull request #289 from virtualtam/make-clean

Makefile: do not call `clean` before `test`

[fix]  #293

Black thumbnails on picture wall after upgrade #293

Makefile: do not call `clean` before `test`

Fixes #288

Modifications:
 - call `make clean` explicitely to clean the workspace
 - add `make clean` to Travis instructions

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

Merge pull request #290 from virtualtam/travis-container

Travis: use the container-based infrastructure

Travis: use the container-based infrastructure

See http://docs.travis-ci.com/user/migrating-from-legacy/

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

Merge pull request #271 from virtualtam/php53

PHP: ensure 5.3 compatibility

PHP: ensure 5.3 compatibility, refactor timezone utilities

Relates to #250

Modifications
 - supported version
   - bump required version from 5.1.0 to 5.3.x
   - update README
   - add PHP 5.3 to Travis environments
 - rewrite array declarations: explicitely use array() instead of []
 - move checkPHPVersion to application/Utils.php
 - move timezone functions to application/TimeZone.php
   - cleanup code
   - improve test coverage

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

Makefile: add a generic rule to run PHPCS against different coding standards

Relates to #95

Usage
 - list available standards
   $ ./vendor/bin/phpcs -i
 - run PHPCS against a given standard
   $ make PHPCS_<standard>

Examples
 $ make PHPCS_PSR1
 $ make PHPCS_Zend

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

Merge pull request #257 from ArthurHoaro/tag-http-referer

Prevent redirection loop everytime we rely on HTTP_REFERER

Prevent redirection loop everytime we rely on HTTP_REFERER:

  * search tag
  * delete tag
  * pagination
  * display privates only
  * delete link
  * new/edit/cancel link return page

Move location generation to Utils.php + unit tests.

Fixes #256

ninja

English mistake cf sebsauvage/Shaarli#221

Fixes warning 'Undefined index: searchtags' while filtering by tags.

Happened if there were not any searchtags already present in the query.

Merge pull request #262 from ArthurHoaro/dup-tags

Avoid tag duplicates

Avoid tag duplicates

 * Prevent duplicate client side with awesomplete
 * Prevent duplicate server side (save_edit processing)

Fixes #261

Merge pull request #268 from ArthurHoaro/dailrss-template

Include the whole <item> in Daily RSS template

Merge pull request #269 from virtualtam/fix/read-config

Fix: data/config.php was not imported