fixed link deletion

When deleting links, the js of the default theme separated ids by an escaped space ('+').
There was a trailing '+' after the ids which led to the php code detecting multiple values
even for single values. In combination with the id '0' this could led to no id found at all
and a resulting php error.

this commit fixes the behavior and adds an additional error handling and trimming to the php code.

Merge pull request #940 from ArthurHoaro/hotfix/empty-urls

Generates a permalink URL if the URL is set to blank

Merge pull request #942 from thewilli/fix-wiki-links

migrated Github wiki links to readthedocs

Merge pull request #946 from thewilli/clean

small code cleanup

code clean: cookie expiration

- unified code style (spaces around operators)
- prevented expiration time to be calculated twice
- replaced tabs with spaces

removed doc and code references to magic quotes

- removed all references to magic quotes
- magic quotes are not supported on PHP >= 5.4 (https://secure.php.net/manual/en/security.magicquotes.php)
- Shaarli does not support PHP < 5.5

migrated Github wiki links to readthedocs

Generates a permalinks URL if the URL is set to blank

Fixes #926

Merge pull request #938 from ArthurHoaro/hotfix/tagcloud-order

Sort tag cloud in alphabetical order

Sort tag cloud in alphabetical order

Fixes #932

Merge pull request #934 from thewilli/fix-note-bookmarklet

fixed note bookmarklet

fixed note bookmarklet

- the double quotes used in the alert() call of the note bookmarklet in the default template collided with the ones of the href tag
- replaced the double quotes with single ones (just like the link bookmarklet)

Merge pull request #928 from virtualtam/documentation/v0.9.1

documentation: update installation instructions for 0.9.1

documentation: update installation instructions for 0.9.1

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

Merge pull request #927 from virtualtam/changelog

Update Changelog for 0.9.1

Update CHANGELOG.md for 0.9.1

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

documentation: fix list formatting

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

Merge pull request #925 from ArthurHoaro/theme/remove-2nd-sort-by

Remove the 2nd green 'sort by' row in link list

Merge pull request #924 from ArthurHoaro/feature/hide-list-search

Hide the search link if no tag has been searched in tag list

Merge pull request #923 from ArthurHoaro/hotfix/untagged

Fix untagged only button

Remove the 2nd green 'sort by' row in link list

It isn't really useful and doesn't look good if there isn't enough tags

Hide the search link if no tag has been searched in tag list

Fix untagged only button

Merge pull request #920 from virtualtam/documentation/rest-3rd

documentation: update 3rd-party resources

documentation: update 3rd-party resources

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

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

Merge pull request #918 from Lucas-C/master

Adding missing empty() as spotted in #889 code review

Adding missing empty() as spotted in #889 code review

Merge pull request #917 from virtualtam/documentation/fixes+improvements

Documentation fixes, improvements and additions

documentation: elaborate on REST API server & client prerequisites

Relates to https://github.com/shaarli/Shaarli/issues/903
Relates to https://github.com/shaarli/Shaarli/issues/905

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

documentation: remove obsolete doc release instructions

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

documentation: add links to example REST API clients

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

documentation: rewrite the REST API PHP client example

Closes https://github.com/shaarli/Shaarli/issues/905
Relates to https://github.com/shaarli/Shaarli/pull/751
See https://shaarli.github.io/api-documentation/

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

documentation: update release download and usage information

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

documentation: remove duplicate "Coding guidelines" page

The information is already present under "Static analysis"

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

documentation: fix rendering and internal references

This is mainly cleanup after switching from Github-flavoured Markdown
rendered by Github Pages, to standard Markdown rendered by MkDocs.

Changed:
- rephrase some section titles

Fixed:
- list rendering (items, sub-items))
- code rendering
- quotes
- dead links

Removed:
- extraneous navigational elements

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

Merge pull request #916 from ArthurHoaro/hotfix/fix-composer-php-version

Fix PHP version configuration in composer.json

Fix PHP version configuration in composer.json

Without this setting, composer would download dependencies depending on the PHP version installed on the system.
E.G. I was getting doctrine/instantiator 1.1, which requires at least PHP 7.1.

Merge pull request #910 from virtualtam/documentation/improvements

Include generated doc in release archives, remove HTML from SCM

Merge pull request #887 from ArthurHoaro/hotfix/dash-tag-rename

Make sure that the tag exists before altering/removing it

Move tag renaming code to LinkDB and unit test it

Merge pull request #889 from Lucas-C/master

Using only one form in linklist.html - fix #885

Merge pull request #913 from virtualtam/fix/readme

readme: fix logo uri

readme: fix logo uri

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

documentation: include generated HTML in release archives

Closes https://github.com/shaarli/Shaarli/issues/908
Relates to https://github.com/shaarli/Shaarli/pull/772

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

documentation: set edit_uri

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

documentation: remove uneeded resources

Relates to https://github.com/shaarli/Shaarli/issues/312
Relates to https://github.com/shaarli/Shaarli/pull/772

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

documentation: remove generated HTML from SCM

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

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

Merge pull request #911 from virtualtam/fix/release/composer

fix: use pinned dependency revisions when generating release archives

fix: use pinned dependency revisions when generating release archives

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

Merge pull request #912 from virtualtam/travis/precise

travis: explicitly set the build dist to `precise`

travis: explicitly set the build dist to `precise`

See https://blog.travis-ci.com/2017-07-11-trusty-as-default-linux-is-coming

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

Using only one form in linklist.html + adding untaggedonly filter - fix #885

Merge pull request #906 from virtualtam/docker/cleanup

docker: remove `dev` image, update documentation

Generate HTML documentation

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

docker: remove `dev` image, update documentation

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

Changed:
- Update Docker image list
- Update Docker documentation structure

Removed:
- Delete Dockerfile and resources for the `dev` image
- Cleanup `doc/` resources

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

Merge pull request #896 from ArthurHoaro/hotfix/firefox-social-subdir

Fix Firefox Social button in the default theme

doc: fix bullet list formatting

https://shaarli.readthedocs.io/en/master/Shaarli-configuration/

Merge pull request #901 from shaarli/fix-900

Remove merge conflict leftover

Merge pull request #899 from smuth4/master

Respect HTTP_X_FORWARDED_HOST

Add tests to cover new server_url behavior

Remove merge conflict leftover

Fixes #900

Respect HTTP_X_FORWARDED_HOST

alongside _PORT and _PROTO
Fixes #879

doc: contributing: remove leftover link to wiki

Merge pull request #772 from nodiscc/rtfd

Generate HTML documentation using MkDocs

makefile: remove [[link]] -> [link](url) conversion logic
all links in documentation have been converted to standard markdown link syntax

make htmlpages

makefile: remove obsolete 'doc' target
official documentation can now be found in doc/md/

doc: release: update doc generation instructions

CONTRIBUTING.md: define new workflow for documentation edition and contributions

doc: replace pandoc requirement with python3-venv

move README contents to doc/md/index.md

make htmlpages

doc: rename "datastore hacks" -> "various hacks", move example scripts to gist.github.com, remove obsolete GH wiki _Sidebar.md

Generate HTML documentation using MkDocs (WIP)

MkDocs is a static site generator geared towards building project documentation.
Documentation source files are written in Markdown, and configured with a single YAML file.

 * http://www.mkdocs.org/
 * http://www.mkdocs.org/user-guide/configuration/

Ref. #312

* remove pandoc-generated HTML documentation
* move markdown doc to doc/md/,
* mkdocs.yml:
  * generate HTML doc in doc/html
  * add pages TOC/ordering
  * use index.md as index page
* Makefile: remove execute permissions from generated files
* Makefile: rewrite htmlpages GFM to markdown conversion using sed:
   awk expression aslo matched '][' which causes invalid output on complex links with images or code blocks
* Add mkdocs.yml to .gitattributes, exclude this file from release archives
* Makefile: rename: htmldoc -> doc_html target
* run make doc: pull latest markdown documentation from wiki
* run make htmlpages: update html documentation

Merge pull request #890 from Lucas-C/taglist-cloud_improvments

Tagcloud/list improvments

Fix Firefox Social button in the default theme

 is no longer required since the JS function is now in .
Also, include the trailing slash in the post URL.

Fixes #895

Tagcloud/list improvments

Merge pull request #894 from Lucas-C/bug893

Fixing "Uncaught TypeError" in shaarli.js - fix #893

Fixing "Uncaught TypeError" in shaarli.js - fix #893

Refactoring of CHANGETAG part to avoid duplicated code

Make sure that the tag exists before altering/removing it

Fixes #886

Merge pull request #882 from ArthurHoaro/feature/edit-timestamp

Add creation date when editing a link

Merge pull request #880 from ArthurHoaro/hotfix/allowed-protocols

Add a whitelist of protocols for URLs

Add creation date when editing a link

Also, alter the title on edition

Fixes #431

Merge pull request #884 from ArthurHoaro/hotfix/bookmarklet-url-limit

Selection is now limited to 2k characters using bookmarklets

Merge pull request #883 from ArthurHoaro/template/visited-link

Display visited links in grey

Bookmarklet size limit: increase to 4500 chars and add an alert warning

Selection is now limited to 2k characters using bookmarklets

to avoid having too large URL

Fixes #528

Display visited links in grey

Fixes #244

Merge pull request #841 from ArthurHoaro/feature/search-no-tag

Empty tag search will look for not tagged links

Merge pull request #881 from ArthurHoaro/feature/note-bookmarklet

Add Note bookmarklet #580

Empty tag search will look for not tagged links

Fixes #784

From now, searching for tags with an empty value will return only not tagged links,
with the search bar showing `x results [not tagged]`.

Note that using the api, the searchtags request parameter must be set to `false` to get the same result.

  - [ ] Update API doc

Add Note bookmarklet #580

Merge pull request #835 from ArthurHoaro/feature/tag-cloud

Adds a taglist view with edit/delete buttons

Tag list: use awesomplete for tag auto completion

Adds a taglist view with edit/delete buttons

  * The tag list can be sort alphabetically or by most used tag
  * Edit/Delete are perform using AJAX, or fallback to 'do=changetag' page
  * New features aren't backported to vintage theme

Add a token available everywhere

Move tagcloud template file to tag.cloud

Add an endpoint to refresh the token

Useful for AJAX requests which burns the token

Merge pull request #819 from ArthurHoaro/feature/multi-delete

Bulk deletion