of Free Subscription Request module and FreeSubscription_viewFreeSubscriptionRequest
view of Free Subscription

Extend`FTPConnector.listFiles` to sort files based on some server-side computed attributes of files.

This uses [`listdir_attr`](http://docs.paramiko.org/en/2.4/api/sftp.html#paramiko.sftp_client.SFTPClient.listdir_attr) method of paramiko. `sort_on` parameter is directly attributes of [`paramiko.sftp_attr.SFTPAttributes`]( http://docs.paramiko.org/en/2.4/api/sftp.html#paramiko.sftp_attr.SFTPAttributes).

At the same time, adds a minimal tests, that will only run if `testSFTPConnection_SFTP_URL` environment variable is set to an URI of a running sftp server supporting password authentication - which means that in our current tests infrastructure it will not run.

/cc @vpelletier @aurel @rafael 

/reviewed-on nexedi/erp5!680

/reviewed-on nexedi/erp5!711

And not the string "None". I thought ZSQLMethods were supposed to complain
when receiving None for non-optional fields...

manage_afterClone clears workflow history, which is precisely what
"duplicate" is trying to avoid.
This fixes a regression introduced in
  commit 7e810d5a
  Author: Vincent Pelletier <vincent@nexedi.com>
  Date:   Tue Feb 20 17:06:46 2018 +0900

      CopySupport: Factorise clipboard-related methods and code paths.

The Diff view in new UI will be using IFrames to display the diff.

Also, add changes in tests:

erp5_ui_test: Update history test according to use of diff gadget
erp5_web_renderjs_ui_test: Update tests according to new Diff view for Historical Revisions
testXHTML: Add Historical Comparison View to the list of listbox with no selection name
testXHTML: Simplify test_html_file and include way to ignore single html file

Language modification is done on another page, accessible from many places for now.

The original need to touch this comment is that it mentions now-removed
Folder_reindexObjectList script. But also:
- it mentions multi-ZEO setups, which are now discouraged if favour of
  using NEO
- it mentions pre-conceived optimisations, which can be wrong and hence
  misleading (up-to-date measures are needed before deciding what to
  optimise)
- it mentions a number of activity nodes which is nowadays exceeded
  without seeing any obvious issue caused by activity table deadlocks
  (which are nevertheless still likely an issue, just not yet breaking
  scalability)

These are now superseded by recursiveReindexObject.

This method does not belong to Folder (nor Base, even as an alias), as
it recurses wihout any limit in a tree whose node count is unknown.
Such generic recursion should either be deferred (split in multiple
transactions, each processing a limited chunk of the tree) or not be
recursive (which of course depends on the need).
There is a single caller left (constructInstance) where the subtree size
is assumed to be reasonable, as it was constructed inside current
transaction. So essentially rename this method so it is only used in such
conditions.