[web.facet] self.w -> w

[web/facet|views/table] bump a bit some heights, split table view methods for easier subclassing

backport stable

cleanup and update some copyright

[migractions] fix change_attribute_type

[migractions] dropped_constraints should accept only droprequired without cstrtype

[form rendering] some refactoring to ease FormRenderer overriding

[deprecation] drop remaining warnings

backport stable

[session] new methods on session to ease access to security/hooks control context managers

[workflow] new 'fire_transition_if_possible' method on IWorkflowable to fire a transition if it's possible

[workflow] new 'incontext-state' view for IWorkflowable entities

[facets/table] vertically group facets by two when possible in order to have a more compact horizontal layout in table filter form

[workflow] add method on IWorkflowable to set initial state of an entity (closes #1686357)

[web test] fix test broken by making cw_source editable by managers

[multi-sources] support for moving an entity from an external source (closes #343818) Original need is to move a user from a ldap source to the system source so we can delete it from ldap without loosing information into the cubicweb instance. We can't wait for the user to be deleted from the ldap since it will be too late then to get back user attributes, so it has to be a manual operation to operate before actual deletion. This makes sense for other sources as well. So the idea is to make the "Any cw_source CWSource" relation editable by managers, and to watch changes of it. We then check the move is possible (ie from an external source to the system source) and do necessary stuff (essentially changing source information and copying data into the system source). Remaining pb is that we don't want the moved entity to be reimported later. To distinguish this state, the trick is to change the associated record in the 'entities' system table with eid=-eid while leaving other fields unchanged, and to add a new record with eid=eid, source='system'. External source will then have consider case where `extid2eid` return a negative eid as 'this entity was known but has been moved, ignore it'. Notice no ui is provided yet, it has currently to be done in a c-c shell.

[repository api] definitly kill usage of word 'pool' to refer to connections set used by a session Also document session's data storage and some other internals. Hopefuly things will get clearer. Closes #1684860: vocabulary confusion in repository code: notion of 'pool'

backport stable

[rql2sql] fix in 7394:d11d88bd08e8 was actually not needed

[rset] close #1683703: rset.get_entity crash on rql query with subquery and aggregat

[rql2sql] fix generated sql for eid comparison. closes #1638695

fix some deprecation warnings

[querier test] test groupby function call / column number

backport stable

[plot views] don't paginate

backport oldstable

[url rewriting] fix nasty bug in build_rset inner function: the same dictionary is used by all queries, hence triggering potentially important bugs on concurrent requests (this dictionary goes back and forth the repository)

[hooks selection optimization] prune hooks when multiple entities are concerned by a hm.call_hooks() (closes: #1672022) the idea is to make a first pass over all the hooks in the registry and to mark put some of them in a disabled list. The disabled hooks are the one which: * are disabled at the session level * have a match_rtype or an is_instance selector which does not match the rtype / etype of the relations / entities for which we are calling the hooks. This works because the repository calls the hooks grouped by rtype or by etype when using the entities or eids_to_from keyword arguments Only hooks with a simple selector or an AndSelector of simple selectors (is_instance and match_rtype) are considered for disabling.

[repo test] datafeed parser's process method should now take raise_on_error argument

[datafeed] propagate raise_on_error to parser's process method