This fixes a regression introduced in
commit f36260d2
("Code clean up in ListBox & EditorField").

Make sure convert is not called in case resource is not convertible.

  - Simplify the code and Minor Optimization for sort

  - Now getDependencyList returns entire list of dependencies even
    if it is already installed.

Follow up changes:
  - Included erp5_full_text_myisam_catalog (dependency of erp5_base)
  - Reimplement sortBusinessTemplates for the new behaviour

This is a follow up change of 63c1f78

- when upgradeSite is called, it will delete orphaned bt is we set
  delete_orphaned to true
- when there is provider calculation in the dependency list, look
  at both installed bt and to all the dependency tree

* fix the state of bt (new, old, present)
* make resolveBusinessTemplateListDependency returning all
  dependencies, even installed ones
* add upgradeSite method to install everything from top level bt
* the code to check version restriction has been removed but
  it should be  moved to another method

Split Discussion Thread reference generator into a separate testable method. Be more strict in generated reference.

Add fieldset class selector which can be used to override fault one when customizing forum's appearance.

'display_list' is also useful at pre conversion level scripts (it allows specify multiple displays for a conversion at once)

Allow pre conversion to happen on any ERP5 Web Page's content  which include relative images (through <img> tags).

This helps writing workflow script because 'listbox' dict does not contain
uids, which makes global 'uids' request value unusable.

Add new properties to the NVP url that is used to make queries on paypal and to the business email that is used define who will receive the payment. Also, the Paypal view was updated to display those properties

Remove line to not check consistency inside navigate method because if the user don't have Manager role, he hasn't permission to run checkConsistency function. Then, if one normal user try to redirect to Paypal page, the page will crash because he don't have permissions.

    - This permit we reuse it everywhere.

 - Arbitrary remove objects and delete workflow history is bad. This code
  was never used anyway.
 - Clean up code which is not used anywhere anymore, legacy from long ago
  implementation.

This reverts commit 20f05949.

While having good naming conventions is good, it's a task to
make it passing. Since it shows no important regression, this
should be commited when this test pass (or it can be enabled first
on only one or two bt, and then on more and more)

Use permanent URL where possible.

In products/ERP5/Document/BusinessTemplate.py, when trying to build
an item, an error occured because item was set to None by clean method.

it can be desired (and reflected in test) that developer needs no selections.