This is newer codemirror with support to jump to line keybinding https://github.com/codemirror/CodeMirror/issues/3030


/cc @isabelle I don't know if this interfere with 53a4b813 

/reviewed-on nexedi/erp5!98

I used this procedure :

git clone https://github.com/codemirror/CodeMirror.git --branch 5.13.4
cd CodeMirror

# rename edit to cm_edit # see 54087b03
find . -not -path "./.git*" -type f -exec sed -i 's/addon\/edit\//addon\/cm_edit\//g' {} +
git mv addon/edit addon/cm_edit
git commit -a -m "ERP5: rename addon/edit -> addon/cm_edit"

# upload to zope ( note that I had to mkdir addon explicitly, otherwise I had some 409 Conflicts and 500 Internal Server Error that I did not investigate)
lftp http://login:password@172.17.0.2:2200/erp5/portal_skins/erp5_code_mirror/ -e 'rm -rf codemirror ; mkdir codemirror ; quit' &&\
lftp http://login:password@172.17.0.2:2200/erp5/portal_skins/erp5_code_mirror/codemirror/ -e 'mkdir addon; mput -d * */* */*/* */*/*/* ; quit '

# run portal_alarms/check_skin_cache to set http_cache

[erp5_web_renderjs_ui] form_dialog: do not update navigation history if action redirect to the same document

form_list search widget layout is like a search engine search form (text input and search button next to it)

I do not see why we would need to check again that value !=
self.getTitle because this check is already done in edit, and edit
supports edit_order and force_update.

following up 3d42e2b9 reference is used
only when title is not defined

when no title, fallback to reference then id

On a module, only display the portal_type.

Should fix jio.get/put on ERP5 non editable documents

cancel & cancel_action require now View permission

Breadcrumb page is removed and merge into the Tab page.

The document title is now usually an history navigation back link.

add test case in erp5_ui_test

/reviewed-on nexedi/erp5!90

/reviewed-on !94

Sanitizing client data would help to prevent such issue.

/reviewed-on nexedi/erp5!92

(400 or 502) + add test case in erp5_ui_test

… 30 minutes difference.

/reviewed-on nexedi/erp5!93

caching_instance_method not used

erp5_simulation_test: as `erp5_default_business_process` is required in some test cases
erp5_simplified_invoicing: as `Invoice Line` should be allowed by `Sale Invoice Transaction`
erp5_configurator_standard_accounting_template: as `default_invoice_rule` business process is needed

defined before using it as argument for isURLAncestorOf.

@vpelletier, this caused a bug, since empty came_from would crash isURLAncestorOf when urlparse is called for this url (https://lab.nexedi.com/nexedi/erp5/blob/master/product/ERP5Type/Core/Folder.py#L517). I think this is a way to solve, I do not know if isURLAncestorOf should be changed also to return something in such situation. 

/reviewed-on nexedi/erp5!89