from Free Subscription Request:

- In case of 'subscription', do not create a new Free Subscription
if one already exists
- In case of 'unsubscription' still create a Free Subscription if none exists
and invalidate it at once
- Allow having variation_category that will be passed to the Free Subscription
- Handle definition of effective_date and expiration_date in Free Subscription better, Rule is:
We prioritise effective_date (or expiration_date) if it exists
Then getStopDate result (stop_date or acquiring from start_date) if it is not None
Finally current date
- Have assertions on variation_category and raise in case of unknown type

start_date is in edit_kw

- Set Free Subscription Request to be of class 'Ticket'
- Add FreeSubscriptionRequest_init to set start_date to now
- In views and listboxed, show (delivery.)start_date as 'Begin Date', that is also sortable

Otherwise, the query will include a pointless "node.uid > 0" condition
(because internal API has no choice but to return a condition) which
confuses the query optimiser and may lead to very suboptimal execution
plans: a range scan of node.uid (catalog as node, node.uid=stock.node_uid)
despite provided account uid list constraining stock.node_uid.

Remove trailing whitespaces.
Make it easier to extend the list of indexed columns without causing
unnecessary diffs (sadly, SQL being SQL, those lines will still cause
extra diffs).

/reviewed-on nexedi/erp5!699

This Merge Request adds ```reachability_workflow``` and adds it to all Coordinate types  (Adress, Email etc.)
so that the can be marked as ```reachable``` or ```unreachable```.
It also adds ```coordinate_interaction_workflow```, so that when a coordinate text changes actions can be triggered.
The default behaviour is that, when coordinate text changes for an 'unreachable' Coordinate, it is marked as reachable.

We already discussed this with @jerome , but I cc @nexedi , since this can break compatibility in projects, because
in the past Coordinates acquired ```validation_state``` by their parent document (Person, Organisation, etc).
So now, if some code checks ```getValidationState``` (though it should be checking on the parent) for Coordinate, or searches Catalog with ```validation_state``` (though it should be ```parent_validation_state```), behaviour will change.

Test Result <a href="https://nexedijs.erp5.net/#/test_result_module/20180608-2D68AF16">here</a> seems good (3 failures in UI tests that seem unrelated)


/reviewed-on nexedi/erp5!691

Because the helper script for invoice, (order, packing lists, etc) printout is using `DateTime.strftime` it does not support dates before 1900.
This is a [documented behavior](https://docs.python.org/2/library/datetime.html#strftime-and-strptime-behavior) in python that dates before 1900 cannot be used.

In these cases it's not supposed to happen, but if by mistake user enter a date before 1900, printing the invoice would fail, which is not good.

These patterns were very simple and did not need strftime.

/reviewed-on nexedi/erp5!694

for Coordinates

Test also checks accessors like getDefaultEmailValidationState.

So that we can use accessors such as getDefaultEmailValidationState on
persons or organisations.

In its generic form it would declare as reachable an unreachable coordinate
when the coordinate_text changes

'coordinate' types.
Also translated_validation_state_title column to
Base_viewBaseFieldLibrary/my_view_mode_contact_listbox

To be used by coordinates (Adress, Email etc.)
so that the can be marked as "reachable" or "unreachable".

/reviewed-on nexedi/erp5!689

is used in Folder_delete, in order to decide if we would delete the object from ZODB
or do workflow transition.

Previous code was not harmful, since isDeletable check above would block
the case of existing workflow with 'delete_action' not possible in current state.
Yet, it was confusing.

/reviewed-on nexedi/erp5!696

Update Colors to patio colors

![Screenshot_2018-06-11_at_17.02.16](/uploads/d0ebc58a32df69a359e3023383fcfec2/Screenshot_2018-06-11_at_17.02.16.png)


/reviewed-on nexedi/erp5!693

See 4bb5d32e

Do not rely on DateTime.strftime which does fails on dates <= 1900.
This simple case does not need all the features of strftime and can be
implemented inline.

It currently fails with such ValueError:
```
year=101 is before 1900; the datetime strftime() methods require year >= 1900
```

Some dialog expect to get the line uid list, even if no cell was directly editable.

Use the same dialog pattern than the mass transition dialog.
Display all documents by using the current query + deletable workflow state.

Use the listbox_uid to get the effective list of document to delete.

This partially revert nexedi/erp5@490af3ba

/reviewed-on !688

Add the portal type "DSN Disenrollment Record" for this