author | Sylvain Thénault <sylvain.thenault@logilab.fr> |
Wed, 20 Jan 2010 11:29:08 +0100 | |
branch | stable |
changeset 4287 | 15499a46c009 |
parent 4212 | ab6573088b4a |
child 4489 | 63128e8b9af9 |
permissions | -rw-r--r-- |
0 | 1 |
"""Core hooks: check schema validity, unsure we are not deleting necessary |
2 |
entities... |
|
3 |
||
4 |
:organization: Logilab |
|
4212
ab6573088b4a
update copyright: welcome 2010
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
4119
diff
changeset
|
5 |
:copyright: 2001-2010 LOGILAB S.A. (Paris, FRANCE), license is LGPL v2. |
0 | 6 |
:contact: http://www.logilab.fr/ -- mailto:contact@logilab.fr |
1977
606923dff11b
big bunch of copyright / docstring update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1802
diff
changeset
|
7 |
:license: GNU Lesser General Public License, v2.1 - http://www.gnu.org/licenses |
0 | 8 |
""" |
9 |
__docformat__ = "restructuredtext en" |
|
10 |
||
1016
26387b836099
use datetime instead of mx.DateTime
sylvain.thenault@logilab.fr
parents:
479
diff
changeset
|
11 |
from datetime import datetime |
0 | 12 |
|
13 |
from cubicweb import UnknownProperty, ValidationError, BadConnectionId |
|
3978
2c95e3033f64
finish yesterday work on rql constraints:
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3887
diff
changeset
|
14 |
from cubicweb.schema import RQLConstraint, RQLUniqueConstraint |
0 | 15 |
from cubicweb.server.pool import Operation, LateOperation, PreCommitOperation |
3773
14fde27a70a2
don't allow that even with cow powers
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3693
diff
changeset
|
16 |
from cubicweb.server.hookhelper import (check_internal_entity, |
2920
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
17 |
get_user_sessions, rproperty) |
0 | 18 |
from cubicweb.server.repository import FTIndexEntityOp |
19 |
||
2605
c4f6a53884ec
[R hooks] use DONT_CHECK_RTYPES_ON_[ADD|DEL] constant, don't check wf related internal relations
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2603
diff
changeset
|
20 |
# special relations that don't have to be checked for integrity, usually |
c4f6a53884ec
[R hooks] use DONT_CHECK_RTYPES_ON_[ADD|DEL] constant, don't check wf related internal relations
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2603
diff
changeset
|
21 |
# because they are handled internally by hooks (so we trust ourselves) |
c4f6a53884ec
[R hooks] use DONT_CHECK_RTYPES_ON_[ADD|DEL] constant, don't check wf related internal relations
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2603
diff
changeset
|
22 |
DONT_CHECK_RTYPES_ON_ADD = set(('owned_by', 'created_by', |
c4f6a53884ec
[R hooks] use DONT_CHECK_RTYPES_ON_[ADD|DEL] constant, don't check wf related internal relations
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2603
diff
changeset
|
23 |
'is', 'is_instance_of', |
c4f6a53884ec
[R hooks] use DONT_CHECK_RTYPES_ON_[ADD|DEL] constant, don't check wf related internal relations
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2603
diff
changeset
|
24 |
'wf_info_for', 'from_state', 'to_state')) |
c4f6a53884ec
[R hooks] use DONT_CHECK_RTYPES_ON_[ADD|DEL] constant, don't check wf related internal relations
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2603
diff
changeset
|
25 |
DONT_CHECK_RTYPES_ON_DEL = set(('is', 'is_instance_of', |
c4f6a53884ec
[R hooks] use DONT_CHECK_RTYPES_ON_[ADD|DEL] constant, don't check wf related internal relations
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2603
diff
changeset
|
26 |
'wf_info_for', 'from_state', 'to_state')) |
c4f6a53884ec
[R hooks] use DONT_CHECK_RTYPES_ON_[ADD|DEL] constant, don't check wf related internal relations
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2603
diff
changeset
|
27 |
|
c4f6a53884ec
[R hooks] use DONT_CHECK_RTYPES_ON_[ADD|DEL] constant, don't check wf related internal relations
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2603
diff
changeset
|
28 |
|
0 | 29 |
def relation_deleted(session, eidfrom, rtype, eidto): |
2101
08003e0354a7
update transaction data api
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
1977
diff
changeset
|
30 |
session.transaction_data.setdefault('pendingrelations', []).append( |
08003e0354a7
update transaction data api
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
1977
diff
changeset
|
31 |
(eidfrom, rtype, eidto)) |
1802
d628defebc17
delete-trailing-whitespace + some copyright update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1398
diff
changeset
|
32 |
|
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
33 |
def eschema_type_eid(session, etype): |
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
34 |
"""get eid of the CWEType entity for the given yams type""" |
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
35 |
eschema = session.repo.schema.eschema(etype) |
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
36 |
# eschema.eid is None if schema has been readen from the filesystem, not |
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
37 |
# from the database (eg during tests) |
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
38 |
if eschema.eid is None: |
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
39 |
eschema.eid = session.unsafe_execute( |
2920
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
40 |
'Any X WHERE X is CWEType, X name %(name)s', |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
41 |
{'name': str(etype)})[0][0] |
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
42 |
return eschema.eid |
0 | 43 |
|
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
44 |
|
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
45 |
# base meta-data handling ###################################################### |
0 | 46 |
|
47 |
def setctime_before_add_entity(session, entity): |
|
48 |
"""before create a new entity -> set creation and modification date |
|
1802
d628defebc17
delete-trailing-whitespace + some copyright update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1398
diff
changeset
|
49 |
|
0 | 50 |
this is a conveniency hook, you shouldn't have to disable it |
51 |
""" |
|
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
52 |
timestamp = datetime.now() |
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
53 |
entity.setdefault('creation_date', timestamp) |
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
54 |
entity.setdefault('modification_date', timestamp) |
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
55 |
if not session.get_shared_data('do-not-insert-cwuri'): |
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
56 |
entity.setdefault('cwuri', u'%seid/%s' % (session.base_url(), entity.eid)) |
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
57 |
|
0 | 58 |
|
59 |
def setmtime_before_update_entity(session, entity): |
|
60 |
"""update an entity -> set modification date""" |
|
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
61 |
entity.setdefault('modification_date', datetime.now()) |
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
62 |
|
1802
d628defebc17
delete-trailing-whitespace + some copyright update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1398
diff
changeset
|
63 |
|
0 | 64 |
class SetCreatorOp(PreCommitOperation): |
1802
d628defebc17
delete-trailing-whitespace + some copyright update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1398
diff
changeset
|
65 |
|
0 | 66 |
def precommit_event(self): |
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
67 |
session = self.session |
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
68 |
if self.entity.eid in session.transaction_data.get('pendingeids', ()): |
0 | 69 |
# entity have been created and deleted in the same transaction |
70 |
return |
|
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
71 |
if not self.entity.created_by: |
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
72 |
session.add_relation(self.entity.eid, 'created_by', session.user.eid) |
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
73 |
|
0 | 74 |
|
75 |
def setowner_after_add_entity(session, entity): |
|
76 |
"""create a new entity -> set owner and creator metadata""" |
|
77 |
asession = session.actual_session() |
|
78 |
if not asession.is_internal_session: |
|
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
79 |
session.add_relation(entity.eid, 'owned_by', asession.user.eid) |
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
80 |
SetCreatorOp(asession, entity=entity) |
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
81 |
|
0 | 82 |
|
83 |
def setis_after_add_entity(session, entity): |
|
84 |
"""create a new entity -> set is relation""" |
|
1250
5c20a7f13c84
new recreate argument to extid2eid when an external source want to recreate entities previously imported with a predictable ext id
sylvain.thenault@logilab.fr
parents:
479
diff
changeset
|
85 |
if hasattr(entity, '_cw_recreating'): |
5c20a7f13c84
new recreate argument to extid2eid when an external source want to recreate entities previously imported with a predictable ext id
sylvain.thenault@logilab.fr
parents:
479
diff
changeset
|
86 |
return |
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
87 |
try: |
3693
92581287726f
[optimisation] insert is / is_instance_of using sql
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3689
diff
changeset
|
88 |
#session.add_relation(entity.eid, 'is', |
92581287726f
[optimisation] insert is / is_instance_of using sql
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3689
diff
changeset
|
89 |
# eschema_type_eid(session, entity.id)) |
92581287726f
[optimisation] insert is / is_instance_of using sql
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3689
diff
changeset
|
90 |
session.system_sql('INSERT INTO is_relation(eid_from,eid_to) VALUES (%s,%s)' |
92581287726f
[optimisation] insert is / is_instance_of using sql
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3689
diff
changeset
|
91 |
% (entity.eid, eschema_type_eid(session, entity.id))) |
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
92 |
except IndexError: |
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
93 |
# during schema serialization, skip |
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
94 |
return |
3693
92581287726f
[optimisation] insert is / is_instance_of using sql
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3689
diff
changeset
|
95 |
for etype in entity.e_schema.ancestors() + [entity.e_schema]: |
92581287726f
[optimisation] insert is / is_instance_of using sql
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3689
diff
changeset
|
96 |
#session.add_relation(entity.eid, 'is_instance_of', |
92581287726f
[optimisation] insert is / is_instance_of using sql
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3689
diff
changeset
|
97 |
# eschema_type_eid(session, etype)) |
92581287726f
[optimisation] insert is / is_instance_of using sql
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3689
diff
changeset
|
98 |
session.system_sql('INSERT INTO is_instance_of_relation(eid_from,eid_to) VALUES (%s,%s)' |
92581287726f
[optimisation] insert is / is_instance_of using sql
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3689
diff
changeset
|
99 |
% (entity.eid, eschema_type_eid(session, etype))) |
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
100 |
|
0 | 101 |
|
102 |
def setowner_after_add_user(session, entity): |
|
103 |
"""when a user has been created, add owned_by relation on itself""" |
|
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
104 |
session.add_relation(entity.eid, 'owned_by', entity.eid) |
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
105 |
|
0 | 106 |
|
107 |
def fti_update_after_add_relation(session, eidfrom, rtype, eidto): |
|
108 |
"""sync fulltext index when relevant relation is added. Reindexing the |
|
109 |
contained entity is enough since it will implicitly reindex the container |
|
110 |
entity. |
|
111 |
""" |
|
112 |
ftcontainer = session.repo.schema.rschema(rtype).fulltext_container |
|
113 |
if ftcontainer == 'subject': |
|
2195
58bef4f707ce
update calls to a deprecated method
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2101
diff
changeset
|
114 |
FTIndexEntityOp(session, entity=session.entity_from_eid(eidto)) |
0 | 115 |
elif ftcontainer == 'object': |
2195
58bef4f707ce
update calls to a deprecated method
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2101
diff
changeset
|
116 |
FTIndexEntityOp(session, entity=session.entity_from_eid(eidfrom)) |
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
117 |
|
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
118 |
|
0 | 119 |
def fti_update_after_delete_relation(session, eidfrom, rtype, eidto): |
120 |
"""sync fulltext index when relevant relation is deleted. Reindexing both |
|
121 |
entities is necessary. |
|
122 |
""" |
|
123 |
if session.repo.schema.rschema(rtype).fulltext_container: |
|
2195
58bef4f707ce
update calls to a deprecated method
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2101
diff
changeset
|
124 |
FTIndexEntityOp(session, entity=session.entity_from_eid(eidto)) |
58bef4f707ce
update calls to a deprecated method
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2101
diff
changeset
|
125 |
FTIndexEntityOp(session, entity=session.entity_from_eid(eidfrom)) |
1802
d628defebc17
delete-trailing-whitespace + some copyright update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1398
diff
changeset
|
126 |
|
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
127 |
|
0 | 128 |
class SyncOwnersOp(PreCommitOperation): |
1802
d628defebc17
delete-trailing-whitespace + some copyright update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1398
diff
changeset
|
129 |
|
0 | 130 |
def precommit_event(self): |
131 |
self.session.unsafe_execute('SET X owned_by U WHERE C owned_by U, C eid %(c)s,' |
|
132 |
'NOT EXISTS(X owned_by U, X eid %(x)s)', |
|
133 |
{'c': self.compositeeid, 'x': self.composedeid}, |
|
134 |
('c', 'x')) |
|
1802
d628defebc17
delete-trailing-whitespace + some copyright update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1398
diff
changeset
|
135 |
|
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
136 |
|
0 | 137 |
def sync_owner_after_add_composite_relation(session, eidfrom, rtype, eidto): |
138 |
"""when adding composite relation, the composed should have the same owners |
|
139 |
has the composite |
|
140 |
""" |
|
141 |
if rtype == 'wf_info_for': |
|
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
142 |
# skip this special composite relation # XXX (syt) why? |
0 | 143 |
return |
144 |
composite = rproperty(session, rtype, eidfrom, eidto, 'composite') |
|
145 |
if composite == 'subject': |
|
146 |
SyncOwnersOp(session, compositeeid=eidfrom, composedeid=eidto) |
|
147 |
elif composite == 'object': |
|
148 |
SyncOwnersOp(session, compositeeid=eidto, composedeid=eidfrom) |
|
1802
d628defebc17
delete-trailing-whitespace + some copyright update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1398
diff
changeset
|
149 |
|
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
150 |
|
0 | 151 |
def _register_metadata_hooks(hm): |
152 |
"""register meta-data related hooks on the hooks manager""" |
|
153 |
hm.register_hook(setctime_before_add_entity, 'before_add_entity', '') |
|
154 |
hm.register_hook(setmtime_before_update_entity, 'before_update_entity', '') |
|
155 |
hm.register_hook(setowner_after_add_entity, 'after_add_entity', '') |
|
156 |
hm.register_hook(sync_owner_after_add_composite_relation, 'after_add_relation', '') |
|
157 |
hm.register_hook(fti_update_after_add_relation, 'after_add_relation', '') |
|
158 |
hm.register_hook(fti_update_after_delete_relation, 'after_delete_relation', '') |
|
159 |
if 'is' in hm.schema: |
|
160 |
hm.register_hook(setis_after_add_entity, 'after_add_entity', '') |
|
1398
5fe84a5f7035
rename internal entity types to have CW prefix instead of E
sylvain.thenault@logilab.fr
parents:
1263
diff
changeset
|
161 |
if 'CWUser' in hm.schema: |
5fe84a5f7035
rename internal entity types to have CW prefix instead of E
sylvain.thenault@logilab.fr
parents:
1263
diff
changeset
|
162 |
hm.register_hook(setowner_after_add_user, 'after_add_entity', 'CWUser') |
1802
d628defebc17
delete-trailing-whitespace + some copyright update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1398
diff
changeset
|
163 |
|
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
164 |
|
0 | 165 |
# core hooks ################################################################## |
1802
d628defebc17
delete-trailing-whitespace + some copyright update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1398
diff
changeset
|
166 |
|
0 | 167 |
class DelayedDeleteOp(PreCommitOperation): |
168 |
"""delete the object of composite relation except if the relation |
|
169 |
has actually been redirected to another composite |
|
170 |
""" |
|
1802
d628defebc17
delete-trailing-whitespace + some copyright update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1398
diff
changeset
|
171 |
|
0 | 172 |
def precommit_event(self): |
173 |
session = self.session |
|
2607
5d889b4928bb
[F hooks] skip new eids as well as pending ones in DelayedDelete operation
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2606
diff
changeset
|
174 |
# don't do anything if the entity is being created or deleted |
5d889b4928bb
[F hooks] skip new eids as well as pending ones in DelayedDelete operation
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2606
diff
changeset
|
175 |
if not (self.eid in session.transaction_data.get('pendingeids', ()) or |
5d889b4928bb
[F hooks] skip new eids as well as pending ones in DelayedDelete operation
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2606
diff
changeset
|
176 |
self.eid in session.transaction_data.get('neweids', ())): |
0 | 177 |
etype = session.describe(self.eid)[0] |
3568
87a7ca9d8ce6
[hooks] don't delete composed of a composite where relation is being removed from the schema
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3554
diff
changeset
|
178 |
if self.role == 'subject': |
87a7ca9d8ce6
[hooks] don't delete composed of a composite where relation is being removed from the schema
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3554
diff
changeset
|
179 |
rql = 'DELETE %s X WHERE X eid %%(x)s, NOT X %s Y' |
87a7ca9d8ce6
[hooks] don't delete composed of a composite where relation is being removed from the schema
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3554
diff
changeset
|
180 |
else: # self.role == 'object': |
87a7ca9d8ce6
[hooks] don't delete composed of a composite where relation is being removed from the schema
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3554
diff
changeset
|
181 |
rql = 'DELETE %s X WHERE X eid %%(x)s, NOT Y %s X' |
87a7ca9d8ce6
[hooks] don't delete composed of a composite where relation is being removed from the schema
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3554
diff
changeset
|
182 |
session.unsafe_execute(rql % (etype, self.rtype), {'x': self.eid}, 'x') |
1802
d628defebc17
delete-trailing-whitespace + some copyright update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1398
diff
changeset
|
183 |
|
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
184 |
|
0 | 185 |
def handle_composite_before_del_relation(session, eidfrom, rtype, eidto): |
186 |
"""delete the object of composite relation""" |
|
3568
87a7ca9d8ce6
[hooks] don't delete composed of a composite where relation is being removed from the schema
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3554
diff
changeset
|
187 |
# if the relation is being delete, don't delete composite's components |
87a7ca9d8ce6
[hooks] don't delete composed of a composite where relation is being removed from the schema
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3554
diff
changeset
|
188 |
# automatically |
87a7ca9d8ce6
[hooks] don't delete composed of a composite where relation is being removed from the schema
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3554
diff
changeset
|
189 |
pendingrdefs = session.transaction_data.get('pendingrdefs', ()) |
87a7ca9d8ce6
[hooks] don't delete composed of a composite where relation is being removed from the schema
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3554
diff
changeset
|
190 |
if (session.describe(eidfrom)[0], rtype, session.describe(eidto)[0]) in pendingrdefs: |
87a7ca9d8ce6
[hooks] don't delete composed of a composite where relation is being removed from the schema
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3554
diff
changeset
|
191 |
return |
0 | 192 |
composite = rproperty(session, rtype, eidfrom, eidto, 'composite') |
193 |
if composite == 'subject': |
|
3568
87a7ca9d8ce6
[hooks] don't delete composed of a composite where relation is being removed from the schema
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3554
diff
changeset
|
194 |
DelayedDeleteOp(session, eid=eidto, rtype=rtype, role='object') |
0 | 195 |
elif composite == 'object': |
3568
87a7ca9d8ce6
[hooks] don't delete composed of a composite where relation is being removed from the schema
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3554
diff
changeset
|
196 |
DelayedDeleteOp(session, eid=eidfrom, rtype=rtype, role='subject') |
0 | 197 |
|
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
198 |
|
0 | 199 |
def before_del_group(session, eid): |
200 |
"""check that we don't remove the owners group""" |
|
201 |
check_internal_entity(session, eid, ('owners',)) |
|
202 |
||
203 |
||
204 |
# schema validation hooks ##################################################### |
|
1802
d628defebc17
delete-trailing-whitespace + some copyright update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1398
diff
changeset
|
205 |
|
0 | 206 |
class CheckConstraintsOperation(LateOperation): |
207 |
"""check a new relation satisfy its constraints |
|
208 |
""" |
|
209 |
def precommit_event(self): |
|
210 |
eidfrom, rtype, eidto = self.rdef |
|
211 |
# first check related entities have not been deleted in the same |
|
212 |
# transaction |
|
2101
08003e0354a7
update transaction data api
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
1977
diff
changeset
|
213 |
pending = self.session.transaction_data.get('pendingeids', ()) |
0 | 214 |
if eidfrom in pending: |
215 |
return |
|
216 |
if eidto in pending: |
|
217 |
return |
|
218 |
for constraint in self.constraints: |
|
219 |
try: |
|
220 |
constraint.repo_check(self.session, eidfrom, rtype, eidto) |
|
221 |
except NotImplementedError: |
|
222 |
self.critical('can\'t check constraint %s, not supported', |
|
223 |
constraint) |
|
1802
d628defebc17
delete-trailing-whitespace + some copyright update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1398
diff
changeset
|
224 |
|
0 | 225 |
def commit_event(self): |
226 |
pass |
|
1802
d628defebc17
delete-trailing-whitespace + some copyright update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1398
diff
changeset
|
227 |
|
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
228 |
|
0 | 229 |
def cstrcheck_after_add_relation(session, eidfrom, rtype, eidto): |
230 |
"""check the relation satisfy its constraints |
|
231 |
||
232 |
this is delayed to a precommit time operation since other relation which |
|
233 |
will make constraint satisfied may be added later. |
|
234 |
""" |
|
3685
6f807804f1d0
no integrity check for super session
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3662
diff
changeset
|
235 |
if session.is_super_session: |
6f807804f1d0
no integrity check for super session
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3662
diff
changeset
|
236 |
return |
0 | 237 |
constraints = rproperty(session, rtype, eidfrom, eidto, 'constraints') |
238 |
if constraints: |
|
3978
2c95e3033f64
finish yesterday work on rql constraints:
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3887
diff
changeset
|
239 |
# XXX get only RQL[Unique]Constraints? |
0 | 240 |
CheckConstraintsOperation(session, constraints=constraints, |
241 |
rdef=(eidfrom, rtype, eidto)) |
|
242 |
||
243 |
def uniquecstrcheck_before_modification(session, entity): |
|
3685
6f807804f1d0
no integrity check for super session
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3662
diff
changeset
|
244 |
if session.is_super_session: |
6f807804f1d0
no integrity check for super session
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3662
diff
changeset
|
245 |
return |
0 | 246 |
eschema = entity.e_schema |
4111
1fda1d356741
use edited_attributes and uniformize attribute schema retreival
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3978
diff
changeset
|
247 |
for attr in entity.edited_attributes: |
4116 | 248 |
val = entity[attr] |
0 | 249 |
if val is None: |
250 |
continue |
|
3689
deb13e88e037
follow yams 0.25 api changes to improve performance
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3685
diff
changeset
|
251 |
if eschema.subjrels[attr].final and \ |
0 | 252 |
eschema.has_unique_values(attr): |
253 |
rql = '%s X WHERE X %s %%(val)s' % (entity.e_schema, attr) |
|
254 |
rset = session.unsafe_execute(rql, {'val': val}) |
|
255 |
if rset and rset[0][0] != entity.eid: |
|
256 |
msg = session._('the value "%s" is already used, use another one') |
|
257 |
raise ValidationError(entity.eid, {attr: msg % val}) |
|
258 |
||
3978
2c95e3033f64
finish yesterday work on rql constraints:
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3887
diff
changeset
|
259 |
|
3554
26e586f3c15c
[schema] make RQL* constraints usable w/ attributes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3528
diff
changeset
|
260 |
def cstrcheck_after_update_attributes(session, entity): |
3685
6f807804f1d0
no integrity check for super session
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3662
diff
changeset
|
261 |
if session.is_super_session: |
6f807804f1d0
no integrity check for super session
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3662
diff
changeset
|
262 |
return |
4119
85bb30fb6d9b
fix name error
Alexandre Fayolle <alexandre.fayolle@logilab.fr>
parents:
4116
diff
changeset
|
263 |
eschema = entity.e_schema |
3554
26e586f3c15c
[schema] make RQL* constraints usable w/ attributes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3528
diff
changeset
|
264 |
for attr in entity.edited_attributes: |
4111
1fda1d356741
use edited_attributes and uniformize attribute schema retreival
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3978
diff
changeset
|
265 |
if eschema.subjrels[attr].final: |
3554
26e586f3c15c
[schema] make RQL* constraints usable w/ attributes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3528
diff
changeset
|
266 |
constraints = [c for c in entity.e_schema.constraints(attr) |
3978
2c95e3033f64
finish yesterday work on rql constraints:
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3887
diff
changeset
|
267 |
if isinstance(c, (RQLConstraint, RQLUniqueConstraint))] |
3554
26e586f3c15c
[schema] make RQL* constraints usable w/ attributes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3528
diff
changeset
|
268 |
if constraints: |
26e586f3c15c
[schema] make RQL* constraints usable w/ attributes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3528
diff
changeset
|
269 |
CheckConstraintsOperation(session, rdef=(entity.eid, attr, None), |
26e586f3c15c
[schema] make RQL* constraints usable w/ attributes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3528
diff
changeset
|
270 |
constraints=constraints) |
26e586f3c15c
[schema] make RQL* constraints usable w/ attributes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3528
diff
changeset
|
271 |
|
0 | 272 |
|
273 |
class CheckRequiredRelationOperation(LateOperation): |
|
274 |
"""checking relation cardinality has to be done after commit in |
|
275 |
case the relation is being replaced |
|
276 |
""" |
|
277 |
eid, rtype = None, None |
|
1802
d628defebc17
delete-trailing-whitespace + some copyright update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1398
diff
changeset
|
278 |
|
0 | 279 |
def precommit_event(self): |
280 |
# recheck pending eids |
|
2101
08003e0354a7
update transaction data api
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
1977
diff
changeset
|
281 |
if self.eid in self.session.transaction_data.get('pendingeids', ()): |
0 | 282 |
return |
3887
130ac9aee402
remove bad invisible character
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
3886
diff
changeset
|
283 |
if self.rtype in self.session.transaction_data.get('pendingrtypes', ()): |
3886
6389e5d6edcb
check we're not deleting the relation before checking its cardinality...
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3859
diff
changeset
|
284 |
return |
0 | 285 |
if self.session.unsafe_execute(*self._rql()).rowcount < 1: |
286 |
etype = self.session.describe(self.eid)[0] |
|
2241
fcf08ac5f8c0
translate schema types
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2195
diff
changeset
|
287 |
_ = self.session._ |
fcf08ac5f8c0
translate schema types
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2195
diff
changeset
|
288 |
msg = _('at least one relation %(rtype)s is required on %(etype)s (%(eid)s)') |
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
289 |
msg %= {'rtype': _(self.rtype), 'etype': _(etype), 'eid': self.eid} |
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
290 |
raise ValidationError(self.eid, {self.rtype: msg}) |
1802
d628defebc17
delete-trailing-whitespace + some copyright update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1398
diff
changeset
|
291 |
|
0 | 292 |
def commit_event(self): |
293 |
pass |
|
1802
d628defebc17
delete-trailing-whitespace + some copyright update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1398
diff
changeset
|
294 |
|
0 | 295 |
def _rql(self): |
296 |
raise NotImplementedError() |
|
1802
d628defebc17
delete-trailing-whitespace + some copyright update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1398
diff
changeset
|
297 |
|
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
298 |
|
0 | 299 |
class CheckSRelationOp(CheckRequiredRelationOperation): |
300 |
"""check required subject relation""" |
|
301 |
def _rql(self): |
|
302 |
return 'Any O WHERE S eid %%(x)s, S %s O' % self.rtype, {'x': self.eid}, 'x' |
|
1802
d628defebc17
delete-trailing-whitespace + some copyright update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1398
diff
changeset
|
303 |
|
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
304 |
|
0 | 305 |
class CheckORelationOp(CheckRequiredRelationOperation): |
306 |
"""check required object relation""" |
|
307 |
def _rql(self): |
|
308 |
return 'Any S WHERE O eid %%(x)s, S %s O' % self.rtype, {'x': self.eid}, 'x' |
|
309 |
||
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
310 |
|
0 | 311 |
def checkrel_if_necessary(session, opcls, rtype, eid): |
312 |
"""check an equivalent operation has not already been added""" |
|
313 |
for op in session.pending_operations: |
|
314 |
if isinstance(op, opcls) and op.rtype == rtype and op.eid == eid: |
|
315 |
break |
|
316 |
else: |
|
317 |
opcls(session, rtype=rtype, eid=eid) |
|
1802
d628defebc17
delete-trailing-whitespace + some copyright update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1398
diff
changeset
|
318 |
|
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
319 |
|
0 | 320 |
def cardinalitycheck_after_add_entity(session, entity): |
321 |
"""check cardinalities are satisfied""" |
|
3685
6f807804f1d0
no integrity check for super session
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3662
diff
changeset
|
322 |
if session.is_super_session: |
6f807804f1d0
no integrity check for super session
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3662
diff
changeset
|
323 |
return |
0 | 324 |
eid = entity.eid |
325 |
for rschema, targetschemas, x in entity.e_schema.relation_definitions(): |
|
326 |
# skip automatically handled relations |
|
2605
c4f6a53884ec
[R hooks] use DONT_CHECK_RTYPES_ON_[ADD|DEL] constant, don't check wf related internal relations
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2603
diff
changeset
|
327 |
if rschema.type in DONT_CHECK_RTYPES_ON_ADD: |
0 | 328 |
continue |
329 |
if x == 'subject': |
|
330 |
subjtype = entity.e_schema |
|
331 |
objtype = targetschemas[0].type |
|
332 |
cardindex = 0 |
|
333 |
opcls = CheckSRelationOp |
|
334 |
else: |
|
335 |
subjtype = targetschemas[0].type |
|
336 |
objtype = entity.e_schema |
|
337 |
cardindex = 1 |
|
338 |
opcls = CheckORelationOp |
|
339 |
card = rschema.rproperty(subjtype, objtype, 'cardinality') |
|
340 |
if card[cardindex] in '1+': |
|
341 |
checkrel_if_necessary(session, opcls, rschema.type, eid) |
|
342 |
||
343 |
def cardinalitycheck_before_del_relation(session, eidfrom, rtype, eidto): |
|
344 |
"""check cardinalities are satisfied""" |
|
3685
6f807804f1d0
no integrity check for super session
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3662
diff
changeset
|
345 |
if session.is_super_session: |
6f807804f1d0
no integrity check for super session
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3662
diff
changeset
|
346 |
return |
2605
c4f6a53884ec
[R hooks] use DONT_CHECK_RTYPES_ON_[ADD|DEL] constant, don't check wf related internal relations
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2603
diff
changeset
|
347 |
if rtype in DONT_CHECK_RTYPES_ON_DEL: |
c4f6a53884ec
[R hooks] use DONT_CHECK_RTYPES_ON_[ADD|DEL] constant, don't check wf related internal relations
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2603
diff
changeset
|
348 |
return |
0 | 349 |
card = rproperty(session, rtype, eidfrom, eidto, 'cardinality') |
2745
0dafa29ace1f
[schema migration] test reproducing pb when deleting a relation definition with mandatory card and without removing the associated rtype + fix
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2695
diff
changeset
|
350 |
pendingrdefs = session.transaction_data.get('pendingrdefs', ()) |
0dafa29ace1f
[schema migration] test reproducing pb when deleting a relation definition with mandatory card and without removing the associated rtype + fix
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2695
diff
changeset
|
351 |
if (session.describe(eidfrom)[0], rtype, session.describe(eidto)[0]) in pendingrdefs: |
0dafa29ace1f
[schema migration] test reproducing pb when deleting a relation definition with mandatory card and without removing the associated rtype + fix
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2695
diff
changeset
|
352 |
return |
2101
08003e0354a7
update transaction data api
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
1977
diff
changeset
|
353 |
pendingeids = session.transaction_data.get('pendingeids', ()) |
0 | 354 |
if card[0] in '1+' and not eidfrom in pendingeids: |
355 |
checkrel_if_necessary(session, CheckSRelationOp, rtype, eidfrom) |
|
356 |
if card[1] in '1+' and not eidto in pendingeids: |
|
357 |
checkrel_if_necessary(session, CheckORelationOp, rtype, eidto) |
|
358 |
||
359 |
||
360 |
def _register_core_hooks(hm): |
|
361 |
hm.register_hook(handle_composite_before_del_relation, 'before_delete_relation', '') |
|
1398
5fe84a5f7035
rename internal entity types to have CW prefix instead of E
sylvain.thenault@logilab.fr
parents:
1263
diff
changeset
|
362 |
hm.register_hook(before_del_group, 'before_delete_entity', 'CWGroup') |
1802
d628defebc17
delete-trailing-whitespace + some copyright update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1398
diff
changeset
|
363 |
|
0 | 364 |
#hm.register_hook(cstrcheck_before_update_entity, 'before_update_entity', '') |
365 |
hm.register_hook(cardinalitycheck_after_add_entity, 'after_add_entity', '') |
|
366 |
hm.register_hook(cardinalitycheck_before_del_relation, 'before_delete_relation', '') |
|
367 |
hm.register_hook(cstrcheck_after_add_relation, 'after_add_relation', '') |
|
368 |
hm.register_hook(uniquecstrcheck_before_modification, 'before_add_entity', '') |
|
369 |
hm.register_hook(uniquecstrcheck_before_modification, 'before_update_entity', '') |
|
3554
26e586f3c15c
[schema] make RQL* constraints usable w/ attributes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3528
diff
changeset
|
370 |
hm.register_hook(cstrcheck_after_update_attributes, 'after_add_entity', '') |
26e586f3c15c
[schema] make RQL* constraints usable w/ attributes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3528
diff
changeset
|
371 |
hm.register_hook(cstrcheck_after_update_attributes, 'after_update_entity', '') |
0 | 372 |
|
373 |
# user/groups synchronisation ################################################# |
|
1802
d628defebc17
delete-trailing-whitespace + some copyright update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1398
diff
changeset
|
374 |
|
0 | 375 |
class GroupOperation(Operation): |
376 |
"""base class for group operation""" |
|
377 |
geid = None |
|
378 |
def __init__(self, session, *args, **kwargs): |
|
379 |
"""override to get the group name before actual groups manipulation: |
|
1802
d628defebc17
delete-trailing-whitespace + some copyright update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1398
diff
changeset
|
380 |
|
0 | 381 |
we may temporarily loose right access during a commit event, so |
382 |
no query should be emitted while comitting |
|
383 |
""" |
|
384 |
rql = 'Any N WHERE G eid %(x)s, G name N' |
|
385 |
result = session.execute(rql, {'x': kwargs['geid']}, 'x', build_descr=False) |
|
386 |
Operation.__init__(self, session, *args, **kwargs) |
|
387 |
self.group = result[0][0] |
|
388 |
||
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
389 |
|
0 | 390 |
class DeleteGroupOp(GroupOperation): |
391 |
"""synchronize user when a in_group relation has been deleted""" |
|
392 |
def commit_event(self): |
|
393 |
"""the observed connections pool has been commited""" |
|
394 |
groups = self.cnxuser.groups |
|
395 |
try: |
|
396 |
groups.remove(self.group) |
|
397 |
except KeyError: |
|
398 |
self.error('user %s not in group %s', self.cnxuser, self.group) |
|
399 |
return |
|
1802
d628defebc17
delete-trailing-whitespace + some copyright update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1398
diff
changeset
|
400 |
|
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
401 |
|
0 | 402 |
def after_del_in_group(session, fromeid, rtype, toeid): |
403 |
"""modify user permission, need to update users""" |
|
404 |
for session_ in get_user_sessions(session.repo, fromeid): |
|
405 |
DeleteGroupOp(session, cnxuser=session_.user, geid=toeid) |
|
406 |
||
1802
d628defebc17
delete-trailing-whitespace + some copyright update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1398
diff
changeset
|
407 |
|
0 | 408 |
class AddGroupOp(GroupOperation): |
409 |
"""synchronize user when a in_group relation has been added""" |
|
410 |
def commit_event(self): |
|
411 |
"""the observed connections pool has been commited""" |
|
412 |
groups = self.cnxuser.groups |
|
413 |
if self.group in groups: |
|
414 |
self.warning('user %s already in group %s', self.cnxuser, |
|
415 |
self.group) |
|
416 |
return |
|
417 |
groups.add(self.group) |
|
418 |
||
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
419 |
|
0 | 420 |
def after_add_in_group(session, fromeid, rtype, toeid): |
421 |
"""modify user permission, need to update users""" |
|
422 |
for session_ in get_user_sessions(session.repo, fromeid): |
|
423 |
AddGroupOp(session, cnxuser=session_.user, geid=toeid) |
|
424 |
||
425 |
||
426 |
class DelUserOp(Operation): |
|
427 |
"""synchronize user when a in_group relation has been added""" |
|
428 |
def __init__(self, session, cnxid): |
|
429 |
self.cnxid = cnxid |
|
430 |
Operation.__init__(self, session) |
|
1802
d628defebc17
delete-trailing-whitespace + some copyright update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1398
diff
changeset
|
431 |
|
0 | 432 |
def commit_event(self): |
433 |
"""the observed connections pool has been commited""" |
|
434 |
try: |
|
435 |
self.repo.close(self.cnxid) |
|
436 |
except BadConnectionId: |
|
437 |
pass # already closed |
|
438 |
||
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
439 |
|
0 | 440 |
def after_del_user(session, eid): |
441 |
"""modify user permission, need to update users""" |
|
442 |
for session_ in get_user_sessions(session.repo, eid): |
|
443 |
DelUserOp(session, session_.id) |
|
1802
d628defebc17
delete-trailing-whitespace + some copyright update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1398
diff
changeset
|
444 |
|
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
445 |
|
0 | 446 |
def _register_usergroup_hooks(hm): |
447 |
"""register user/group related hooks on the hooks manager""" |
|
1398
5fe84a5f7035
rename internal entity types to have CW prefix instead of E
sylvain.thenault@logilab.fr
parents:
1263
diff
changeset
|
448 |
hm.register_hook(after_del_user, 'after_delete_entity', 'CWUser') |
0 | 449 |
hm.register_hook(after_add_in_group, 'after_add_relation', 'in_group') |
450 |
hm.register_hook(after_del_in_group, 'after_delete_relation', 'in_group') |
|
451 |
||
452 |
||
453 |
# workflow handling ########################################################### |
|
454 |
||
2992
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
455 |
from cubicweb.entities.wfobjs import WorkflowTransition, WorkflowException |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
456 |
|
2949
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
457 |
def _change_state(session, x, oldstate, newstate): |
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
458 |
nocheck = session.transaction_data.setdefault('skip-security', set()) |
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
459 |
nocheck.add((x, 'in_state', oldstate)) |
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
460 |
nocheck.add((x, 'in_state', newstate)) |
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
461 |
# delete previous state first in case we're using a super session |
3039
7d5a4d27d052
[multi-sources] in ms config, don't try to delete previous state in some condition
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2992
diff
changeset
|
462 |
fromsource = session.describe(x)[1] |
7d5a4d27d052
[multi-sources] in ms config, don't try to delete previous state in some condition
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2992
diff
changeset
|
463 |
# don't try to remove previous state if in_state isn't stored in the system |
7d5a4d27d052
[multi-sources] in ms config, don't try to delete previous state in some condition
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2992
diff
changeset
|
464 |
# source |
7d5a4d27d052
[multi-sources] in ms config, don't try to delete previous state in some condition
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2992
diff
changeset
|
465 |
if fromsource == 'system' or \ |
7d5a4d27d052
[multi-sources] in ms config, don't try to delete previous state in some condition
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2992
diff
changeset
|
466 |
not session.repo.sources_by_uri[fromsource].support_relation('in_state'): |
7d5a4d27d052
[multi-sources] in ms config, don't try to delete previous state in some condition
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2992
diff
changeset
|
467 |
session.delete_relation(x, 'in_state', oldstate) |
2949
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
468 |
session.add_relation(x, 'in_state', newstate) |
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
469 |
|
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
470 |
|
3528
77a69de16709
support for automatic transition
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3508
diff
changeset
|
471 |
class FireAutotransitionOp(PreCommitOperation): |
77a69de16709
support for automatic transition
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3508
diff
changeset
|
472 |
"""try to fire auto transition after state changes""" |
77a69de16709
support for automatic transition
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3508
diff
changeset
|
473 |
|
77a69de16709
support for automatic transition
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3508
diff
changeset
|
474 |
def precommit_event(self): |
77a69de16709
support for automatic transition
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3508
diff
changeset
|
475 |
session = self.session |
77a69de16709
support for automatic transition
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3508
diff
changeset
|
476 |
entity = self.entity |
77a69de16709
support for automatic transition
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3508
diff
changeset
|
477 |
autotrs = list(entity.possible_transitions('auto')) |
77a69de16709
support for automatic transition
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3508
diff
changeset
|
478 |
if autotrs: |
77a69de16709
support for automatic transition
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3508
diff
changeset
|
479 |
assert len(autotrs) == 1 |
77a69de16709
support for automatic transition
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3508
diff
changeset
|
480 |
entity.fire_transition(autotrs[0]) |
77a69de16709
support for automatic transition
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3508
diff
changeset
|
481 |
|
77a69de16709
support for automatic transition
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3508
diff
changeset
|
482 |
|
2920
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
483 |
def before_add_trinfo(session, entity): |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
484 |
"""check the transition is allowed, add missing information. Expect that: |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
485 |
* wf_info_for inlined relation is set |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
486 |
* by_transition or to_state (managers only) inlined relation is set |
0 | 487 |
""" |
2920
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
488 |
# first retreive entity to which the state change apply |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
489 |
try: |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
490 |
foreid = entity['wf_info_for'] |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
491 |
except KeyError: |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
492 |
msg = session._('mandatory relation') |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
493 |
raise ValidationError(entity.eid, {'wf_info_for': msg}) |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
494 |
forentity = session.entity_from_eid(foreid) |
2992
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
495 |
# then check it has a workflow set, unless we're in the process of changing |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
496 |
# entity's workflow |
2978
d8c5ad14ab8e
[wf]Â distinguish main workflow / current workflow
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2949
diff
changeset
|
497 |
if session.transaction_data.get((forentity.eid, 'customwf')): |
d8c5ad14ab8e
[wf]Â distinguish main workflow / current workflow
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2949
diff
changeset
|
498 |
wfeid = session.transaction_data[(forentity.eid, 'customwf')] |
d8c5ad14ab8e
[wf]Â distinguish main workflow / current workflow
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2949
diff
changeset
|
499 |
wf = session.entity_from_eid(wfeid) |
d8c5ad14ab8e
[wf]Â distinguish main workflow / current workflow
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2949
diff
changeset
|
500 |
else: |
d8c5ad14ab8e
[wf]Â distinguish main workflow / current workflow
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2949
diff
changeset
|
501 |
wf = forentity.current_workflow |
2920
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
502 |
if wf is None: |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
503 |
msg = session._('related entity has no workflow set') |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
504 |
raise ValidationError(entity.eid, {None: msg}) |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
505 |
# then check it has a state set |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
506 |
fromstate = forentity.current_state |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
507 |
if fromstate is None: |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
508 |
msg = session._('related entity has no state') |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
509 |
raise ValidationError(entity.eid, {None: msg}) |
2992
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
510 |
# True if we are coming back from subworkflow |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
511 |
swtr = session.transaction_data.pop((forentity.eid, 'subwfentrytr'), None) |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
512 |
cowpowers = session.is_super_session or 'managers' in session.user.groups |
2920
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
513 |
# no investigate the requested state change... |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
514 |
try: |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
515 |
treid = entity['by_transition'] |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
516 |
except KeyError: |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
517 |
# no transition set, check user is a manager and destination state is |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
518 |
# specified (and valid) |
2992
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
519 |
if not cowpowers: |
2920
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
520 |
msg = session._('mandatory relation') |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
521 |
raise ValidationError(entity.eid, {'by_transition': msg}) |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
522 |
deststateeid = entity.get('to_state') |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
523 |
if not deststateeid: |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
524 |
msg = session._('mandatory relation') |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
525 |
raise ValidationError(entity.eid, {'by_transition': msg}) |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
526 |
deststate = wf.state_by_eid(deststateeid) |
3773
14fde27a70a2
don't allow that even with cow powers
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3693
diff
changeset
|
527 |
if deststate is None: |
14fde27a70a2
don't allow that even with cow powers
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3693
diff
changeset
|
528 |
msg = entity.req._("state doesn't belong to entity's current workflow") |
2920
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
529 |
raise ValidationError(entity.eid, {'to_state': msg}) |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
530 |
else: |
2992
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
531 |
# check transition is valid and allowed, unless we're coming back from |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
532 |
# subworkflow |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
533 |
tr = session.entity_from_eid(treid) |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
534 |
if swtr is None: |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
535 |
if tr is None: |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
536 |
msg = session._("transition doesn't belong to entity's workflow") |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
537 |
raise ValidationError(entity.eid, {'by_transition': msg}) |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
538 |
if not tr.has_input_state(fromstate): |
3859
85e6ba89837a
more helpful error message
Aurelien Campeas <aurelien.campeas@logilab.fr>
parents:
3773
diff
changeset
|
539 |
_ = session._ |
4098
eb4722dd5f86
[i18n] use named substitutions for proper localization
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
3978
diff
changeset
|
540 |
msg = _("transition %(tr)s isn't allowed from %(st)s") % {'tr': _(tr.name), |
eb4722dd5f86
[i18n] use named substitutions for proper localization
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
3978
diff
changeset
|
541 |
'st': _(fromstate.name), |
eb4722dd5f86
[i18n] use named substitutions for proper localization
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
3978
diff
changeset
|
542 |
} |
2992
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
543 |
raise ValidationError(entity.eid, {'by_transition': msg}) |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
544 |
if not tr.may_be_fired(foreid): |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
545 |
msg = session._("transition may not be fired") |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
546 |
raise ValidationError(entity.eid, {'by_transition': msg}) |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
547 |
if entity.get('to_state'): |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
548 |
deststateeid = entity['to_state'] |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
549 |
if not cowpowers and deststateeid != tr.destination().eid: |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
550 |
msg = session._("transition isn't allowed") |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
551 |
raise ValidationError(entity.eid, {'by_transition': msg}) |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
552 |
if swtr is None: |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
553 |
deststate = session.entity_from_eid(deststateeid) |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
554 |
if not cowpowers and deststate is None: |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
555 |
msg = entity.req._("state doesn't belong to entity's workflow") |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
556 |
raise ValidationError(entity.eid, {'to_state': msg}) |
0 | 557 |
else: |
2992
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
558 |
deststateeid = tr.destination().eid |
2920
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
559 |
# everything is ok, add missing information on the trinfo entity |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
560 |
entity['from_state'] = fromstate.eid |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
561 |
entity['to_state'] = deststateeid |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
562 |
nocheck = session.transaction_data.setdefault('skip-security', set()) |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
563 |
nocheck.add((entity.eid, 'from_state', fromstate.eid)) |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
564 |
nocheck.add((entity.eid, 'to_state', deststateeid)) |
3528
77a69de16709
support for automatic transition
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3508
diff
changeset
|
565 |
FireAutotransitionOp(session, entity=forentity) |
77a69de16709
support for automatic transition
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3508
diff
changeset
|
566 |
|
2920
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
567 |
|
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
568 |
def after_add_trinfo(session, entity): |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
569 |
"""change related entity state""" |
2949
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
570 |
_change_state(session, entity['wf_info_for'], |
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
571 |
entity['from_state'], entity['to_state']) |
2992
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
572 |
forentity = session.entity_from_eid(entity['wf_info_for']) |
3568
87a7ca9d8ce6
[hooks] don't delete composed of a composite where relation is being removed from the schema
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3554
diff
changeset
|
573 |
assert forentity.current_state.eid == entity['to_state'], ( |
87a7ca9d8ce6
[hooks] don't delete composed of a composite where relation is being removed from the schema
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
3554
diff
changeset
|
574 |
forentity.eid, forentity.current_state.name) |
2992
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
575 |
if forentity.main_workflow.eid != forentity.current_workflow.eid: |
3662
9eeadad82d93
defer trinfo creation when cancelling subworkflow
Aurelien Campeas <aurelien.campeas@logilab.fr>
parents:
3628
diff
changeset
|
576 |
SubWorkflowExitOp(session, forentity=forentity, trinfo=entity) |
9eeadad82d93
defer trinfo creation when cancelling subworkflow
Aurelien Campeas <aurelien.campeas@logilab.fr>
parents:
3628
diff
changeset
|
577 |
|
9eeadad82d93
defer trinfo creation when cancelling subworkflow
Aurelien Campeas <aurelien.campeas@logilab.fr>
parents:
3628
diff
changeset
|
578 |
class SubWorkflowExitOp(PreCommitOperation): |
9eeadad82d93
defer trinfo creation when cancelling subworkflow
Aurelien Campeas <aurelien.campeas@logilab.fr>
parents:
3628
diff
changeset
|
579 |
def precommit_event(self): |
9eeadad82d93
defer trinfo creation when cancelling subworkflow
Aurelien Campeas <aurelien.campeas@logilab.fr>
parents:
3628
diff
changeset
|
580 |
session = self.session |
9eeadad82d93
defer trinfo creation when cancelling subworkflow
Aurelien Campeas <aurelien.campeas@logilab.fr>
parents:
3628
diff
changeset
|
581 |
forentity = self.forentity |
9eeadad82d93
defer trinfo creation when cancelling subworkflow
Aurelien Campeas <aurelien.campeas@logilab.fr>
parents:
3628
diff
changeset
|
582 |
trinfo = self.trinfo |
2992
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
583 |
# we're in a subworkflow, check if we've reached an exit point |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
584 |
wftr = forentity.subworkflow_input_transition() |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
585 |
if wftr is None: |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
586 |
# inconsistency detected |
3662
9eeadad82d93
defer trinfo creation when cancelling subworkflow
Aurelien Campeas <aurelien.campeas@logilab.fr>
parents:
3628
diff
changeset
|
587 |
msg = session._("state doesn't belong to entity's current workflow") |
9eeadad82d93
defer trinfo creation when cancelling subworkflow
Aurelien Campeas <aurelien.campeas@logilab.fr>
parents:
3628
diff
changeset
|
588 |
raise ValidationError(self.trinfo.eid, {'to_state': msg}) |
9eeadad82d93
defer trinfo creation when cancelling subworkflow
Aurelien Campeas <aurelien.campeas@logilab.fr>
parents:
3628
diff
changeset
|
589 |
tostate = wftr.get_exit_point(forentity, trinfo['to_state']) |
2992
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
590 |
if tostate is not None: |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
591 |
# reached an exit point |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
592 |
msg = session._('exiting from subworkflow %s') |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
593 |
msg %= session._(forentity.current_workflow.name) |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
594 |
session.transaction_data[(forentity.eid, 'subwfentrytr')] = True |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
595 |
# XXX iirk |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
596 |
req = forentity.req |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
597 |
forentity.req = session.super_session |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
598 |
try: |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
599 |
trinfo = forentity.change_state(tostate, msg, u'text/plain', |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
600 |
tr=wftr) |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
601 |
finally: |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
602 |
forentity.req = req |
0 | 603 |
|
604 |
||
605 |
class SetInitialStateOp(PreCommitOperation): |
|
606 |
"""make initial state be a default state""" |
|
607 |
||
608 |
def precommit_event(self): |
|
609 |
session = self.session |
|
610 |
entity = self.entity |
|
611 |
# if there is an initial state and the entity's state is not set, |
|
612 |
# use the initial state as a default state |
|
2101
08003e0354a7
update transaction data api
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
1977
diff
changeset
|
613 |
pendingeids = session.transaction_data.get('pendingeids', ()) |
2920
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
614 |
if not entity.eid in pendingeids and not entity.in_state and \ |
2978
d8c5ad14ab8e
[wf]Â distinguish main workflow / current workflow
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2949
diff
changeset
|
615 |
entity.main_workflow: |
d8c5ad14ab8e
[wf]Â distinguish main workflow / current workflow
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2949
diff
changeset
|
616 |
state = entity.main_workflow.initial |
2920
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
617 |
if state: |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
618 |
# use super session to by-pass security checks |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
619 |
session.super_session.add_relation(entity.eid, 'in_state', |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
620 |
state.eid) |
0 | 621 |
|
622 |
||
623 |
def set_initial_state_after_add(session, entity): |
|
624 |
SetInitialStateOp(session, entity=entity) |
|
1802
d628defebc17
delete-trailing-whitespace + some copyright update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1398
diff
changeset
|
625 |
|
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
626 |
|
2983
b458631fe347
[wf] test in_state consistency
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2978
diff
changeset
|
627 |
def before_add_in_state(session, eidfrom, rtype, eidto): |
2992
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
628 |
"""check state apply, in case of direct in_state change using unsafe_execute |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
629 |
""" |
3508
d9915224f9a5
[B] server: fix initial data structure in session.transaction_data
Julien Jehannet <julien.jehannet@logilab.fr>
parents:
3071
diff
changeset
|
630 |
nocheck = session.transaction_data.setdefault('skip-security', set()) |
2983
b458631fe347
[wf] test in_state consistency
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2978
diff
changeset
|
631 |
if (eidfrom, 'in_state', eidto) in nocheck: |
b458631fe347
[wf] test in_state consistency
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2978
diff
changeset
|
632 |
# state changed through TrInfo insertion, so we already know it's ok |
b458631fe347
[wf] test in_state consistency
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2978
diff
changeset
|
633 |
return |
b458631fe347
[wf] test in_state consistency
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2978
diff
changeset
|
634 |
entity = session.entity_from_eid(eidfrom) |
b458631fe347
[wf] test in_state consistency
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2978
diff
changeset
|
635 |
mainwf = entity.main_workflow |
b458631fe347
[wf] test in_state consistency
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2978
diff
changeset
|
636 |
if mainwf is None: |
b458631fe347
[wf] test in_state consistency
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2978
diff
changeset
|
637 |
msg = session._('entity has no workflow set') |
b458631fe347
[wf] test in_state consistency
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2978
diff
changeset
|
638 |
raise ValidationError(entity.eid, {None: msg}) |
b458631fe347
[wf] test in_state consistency
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2978
diff
changeset
|
639 |
for wf in mainwf.iter_workflows(): |
b458631fe347
[wf] test in_state consistency
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2978
diff
changeset
|
640 |
if wf.state_by_eid(eidto): |
b458631fe347
[wf] test in_state consistency
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2978
diff
changeset
|
641 |
break |
b458631fe347
[wf] test in_state consistency
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2978
diff
changeset
|
642 |
else: |
b458631fe347
[wf] test in_state consistency
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2978
diff
changeset
|
643 |
msg = session._("state doesn't belong to entity's workflow. You may " |
b458631fe347
[wf] test in_state consistency
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2978
diff
changeset
|
644 |
"want to set a custom workflow for this entity first.") |
b458631fe347
[wf] test in_state consistency
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2978
diff
changeset
|
645 |
raise ValidationError(eidfrom, {'in_state': msg}) |
2992
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
646 |
if entity.current_workflow and wf.eid != entity.current_workflow.eid: |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
647 |
msg = session._("state doesn't belong to entity's current workflow") |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
648 |
raise ValidationError(eidfrom, {'in_state': msg}) |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
649 |
|
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
650 |
|
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
651 |
class CheckTrExitPoint(PreCommitOperation): |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
652 |
|
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
653 |
def precommit_event(self): |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
654 |
tr = self.session.entity_from_eid(self.treid) |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
655 |
outputs = set() |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
656 |
for ep in tr.subworkflow_exit: |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
657 |
if ep.subwf_state.eid in outputs: |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
658 |
msg = self.session._("can't have multiple exits on the same state") |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
659 |
raise ValidationError(self.treid, {'subworkflow_exit': msg}) |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
660 |
outputs.add(ep.subwf_state.eid) |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
661 |
|
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
662 |
|
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
663 |
def after_add_subworkflow_exit(session, eidfrom, rtype, eidto): |
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
664 |
CheckTrExitPoint(session, treid=eidfrom) |
2983
b458631fe347
[wf] test in_state consistency
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2978
diff
changeset
|
665 |
|
2949
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
666 |
|
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
667 |
class WorkflowChangedOp(PreCommitOperation): |
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
668 |
"""fix entity current state when changing its workflow""" |
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
669 |
|
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
670 |
def precommit_event(self): |
2985 | 671 |
# notice that enforcement that new workflow apply to the entity's type is |
672 |
# done by schema rule, no need to check it here |
|
2949
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
673 |
session = self.session |
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
674 |
pendingeids = session.transaction_data.get('pendingeids', ()) |
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
675 |
if self.eid in pendingeids: |
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
676 |
return |
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
677 |
entity = session.entity_from_eid(self.eid) |
2978
d8c5ad14ab8e
[wf]Â distinguish main workflow / current workflow
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2949
diff
changeset
|
678 |
# check custom workflow has not been rechanged to another one in the same |
d8c5ad14ab8e
[wf]Â distinguish main workflow / current workflow
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2949
diff
changeset
|
679 |
# transaction |
d8c5ad14ab8e
[wf]Â distinguish main workflow / current workflow
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2949
diff
changeset
|
680 |
mainwf = entity.main_workflow |
d8c5ad14ab8e
[wf]Â distinguish main workflow / current workflow
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2949
diff
changeset
|
681 |
if mainwf.eid == self.wfeid: |
d8c5ad14ab8e
[wf]Â distinguish main workflow / current workflow
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2949
diff
changeset
|
682 |
deststate = mainwf.initial |
2949
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
683 |
if not deststate: |
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
684 |
msg = session._('workflow has no initial state') |
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
685 |
raise ValidationError(entity.eid, {'custom_workflow': msg}) |
2978
d8c5ad14ab8e
[wf]Â distinguish main workflow / current workflow
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2949
diff
changeset
|
686 |
if mainwf.state_by_eid(entity.current_state.eid): |
2949
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
687 |
# nothing to do |
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
688 |
return |
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
689 |
# if there are no history, simply go to new workflow's initial state |
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
690 |
if not entity.workflow_history: |
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
691 |
if entity.current_state.eid != deststate.eid: |
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
692 |
_change_state(session, entity.eid, |
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
693 |
entity.current_state.eid, deststate.eid) |
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
694 |
return |
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
695 |
msg = session._('workflow changed to "%s"') |
2978
d8c5ad14ab8e
[wf]Â distinguish main workflow / current workflow
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2949
diff
changeset
|
696 |
msg %= session._(mainwf.name) |
d8c5ad14ab8e
[wf]Â distinguish main workflow / current workflow
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2949
diff
changeset
|
697 |
session.transaction_data[(entity.eid, 'customwf')] = self.wfeid |
2992
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
698 |
entity.change_state(deststate, msg, u'text/plain') |
2949
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
699 |
|
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
700 |
|
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
701 |
def set_custom_workflow(session, eidfrom, rtype, eidto): |
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
702 |
WorkflowChangedOp(session, eid=eidfrom, wfeid=eidto) |
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
703 |
|
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
704 |
|
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
705 |
def del_custom_workflow(session, eidfrom, rtype, eidto): |
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
706 |
entity = session.entity_from_eid(eidfrom) |
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
707 |
typewf = entity.cwetype_workflow() |
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
708 |
if typewf is not None: |
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
709 |
WorkflowChangedOp(session, eid=eidfrom, wfeid=typewf.eid) |
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
710 |
|
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
711 |
|
2920
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
712 |
def after_del_workflow(session, eid): |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
713 |
# workflow cleanup |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
714 |
session.execute('DELETE State X WHERE NOT X state_of Y') |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
715 |
session.execute('DELETE Transition X WHERE NOT X transition_of Y') |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
716 |
|
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
717 |
|
0 | 718 |
def _register_wf_hooks(hm): |
719 |
"""register workflow related hooks on the hooks manager""" |
|
720 |
if 'in_state' in hm.schema: |
|
2920
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
721 |
hm.register_hook(before_add_trinfo, 'before_add_entity', 'TrInfo') |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
722 |
hm.register_hook(after_add_trinfo, 'after_add_entity', 'TrInfo') |
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
723 |
#hm.register_hook(relation_deleted, 'before_delete_relation', 'in_state') |
0 | 724 |
for eschema in hm.schema.entities(): |
725 |
if 'in_state' in eschema.subject_relations(): |
|
726 |
hm.register_hook(set_initial_state_after_add, 'after_add_entity', |
|
727 |
str(eschema)) |
|
2949
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
728 |
hm.register_hook(set_custom_workflow, 'after_add_relation', 'custom_workflow') |
a2aa2c51f3be
test and implements workflow changes
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2920
diff
changeset
|
729 |
hm.register_hook(del_custom_workflow, 'after_delete_relation', 'custom_workflow') |
2920
64322aa83a1d
start a new workflow engine
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2909
diff
changeset
|
730 |
hm.register_hook(after_del_workflow, 'after_delete_entity', 'Workflow') |
2983
b458631fe347
[wf] test in_state consistency
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2978
diff
changeset
|
731 |
hm.register_hook(before_add_in_state, 'before_add_relation', 'in_state') |
2992
a5b8bf107a1a
[wf] test and hooks for WorkflowTransition support
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2985
diff
changeset
|
732 |
hm.register_hook(after_add_subworkflow_exit, 'after_add_relation', 'subworkflow_exit') |
0 | 733 |
|
734 |
||
1398
5fe84a5f7035
rename internal entity types to have CW prefix instead of E
sylvain.thenault@logilab.fr
parents:
1263
diff
changeset
|
735 |
# CWProperty hooks ############################################################# |
0 | 736 |
|
737 |
||
1398
5fe84a5f7035
rename internal entity types to have CW prefix instead of E
sylvain.thenault@logilab.fr
parents:
1263
diff
changeset
|
738 |
class DelCWPropertyOp(Operation): |
0 | 739 |
"""a user's custom properties has been deleted""" |
1802
d628defebc17
delete-trailing-whitespace + some copyright update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1398
diff
changeset
|
740 |
|
0 | 741 |
def commit_event(self): |
742 |
"""the observed connections pool has been commited""" |
|
743 |
try: |
|
744 |
del self.epropdict[self.key] |
|
745 |
except KeyError: |
|
746 |
self.error('%s has no associated value', self.key) |
|
747 |
||
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
748 |
|
1398
5fe84a5f7035
rename internal entity types to have CW prefix instead of E
sylvain.thenault@logilab.fr
parents:
1263
diff
changeset
|
749 |
class ChangeCWPropertyOp(Operation): |
0 | 750 |
"""a user's custom properties has been added/changed""" |
1802
d628defebc17
delete-trailing-whitespace + some copyright update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1398
diff
changeset
|
751 |
|
0 | 752 |
def commit_event(self): |
753 |
"""the observed connections pool has been commited""" |
|
754 |
self.epropdict[self.key] = self.value |
|
755 |
||
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
756 |
|
1398
5fe84a5f7035
rename internal entity types to have CW prefix instead of E
sylvain.thenault@logilab.fr
parents:
1263
diff
changeset
|
757 |
class AddCWPropertyOp(Operation): |
0 | 758 |
"""a user's custom properties has been added/changed""" |
1802
d628defebc17
delete-trailing-whitespace + some copyright update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1398
diff
changeset
|
759 |
|
0 | 760 |
def commit_event(self): |
761 |
"""the observed connections pool has been commited""" |
|
762 |
eprop = self.eprop |
|
763 |
if not eprop.for_user: |
|
764 |
self.repo.vreg.eprop_values[eprop.pkey] = eprop.value |
|
1398
5fe84a5f7035
rename internal entity types to have CW prefix instead of E
sylvain.thenault@logilab.fr
parents:
1263
diff
changeset
|
765 |
# if for_user is set, update is handled by a ChangeCWPropertyOp operation |
0 | 766 |
|
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
767 |
|
0 | 768 |
def after_add_eproperty(session, entity): |
769 |
key, value = entity.pkey, entity.value |
|
770 |
try: |
|
771 |
value = session.vreg.typed_value(key, value) |
|
772 |
except UnknownProperty: |
|
773 |
raise ValidationError(entity.eid, {'pkey': session._('unknown property key')}) |
|
774 |
except ValueError, ex: |
|
775 |
raise ValidationError(entity.eid, {'value': session._(str(ex))}) |
|
776 |
if not session.user.matching_groups('managers'): |
|
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
777 |
session.add_relation(entity.eid, 'for_user', session.user.eid) |
0 | 778 |
else: |
1398
5fe84a5f7035
rename internal entity types to have CW prefix instead of E
sylvain.thenault@logilab.fr
parents:
1263
diff
changeset
|
779 |
AddCWPropertyOp(session, eprop=entity) |
1802
d628defebc17
delete-trailing-whitespace + some copyright update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1398
diff
changeset
|
780 |
|
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
781 |
|
0 | 782 |
def after_update_eproperty(session, entity): |
2695
56439c45781c
[hooks] check key or value is actually edited
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2607
diff
changeset
|
783 |
if not ('pkey' in entity.edited_attributes or |
56439c45781c
[hooks] check key or value is actually edited
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2607
diff
changeset
|
784 |
'value' in entity.edited_attributes): |
56439c45781c
[hooks] check key or value is actually edited
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2607
diff
changeset
|
785 |
return |
0 | 786 |
key, value = entity.pkey, entity.value |
787 |
try: |
|
788 |
value = session.vreg.typed_value(key, value) |
|
789 |
except UnknownProperty: |
|
790 |
return |
|
791 |
except ValueError, ex: |
|
792 |
raise ValidationError(entity.eid, {'value': session._(str(ex))}) |
|
793 |
if entity.for_user: |
|
794 |
for session_ in get_user_sessions(session.repo, entity.for_user[0].eid): |
|
1398
5fe84a5f7035
rename internal entity types to have CW prefix instead of E
sylvain.thenault@logilab.fr
parents:
1263
diff
changeset
|
795 |
ChangeCWPropertyOp(session, epropdict=session_.user.properties, |
0 | 796 |
key=key, value=value) |
797 |
else: |
|
798 |
# site wide properties |
|
1398
5fe84a5f7035
rename internal entity types to have CW prefix instead of E
sylvain.thenault@logilab.fr
parents:
1263
diff
changeset
|
799 |
ChangeCWPropertyOp(session, epropdict=session.vreg.eprop_values, |
0 | 800 |
key=key, value=value) |
1802
d628defebc17
delete-trailing-whitespace + some copyright update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1398
diff
changeset
|
801 |
|
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
802 |
|
0 | 803 |
def before_del_eproperty(session, eid): |
2101
08003e0354a7
update transaction data api
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
1977
diff
changeset
|
804 |
for eidfrom, rtype, eidto in session.transaction_data.get('pendingrelations', ()): |
0 | 805 |
if rtype == 'for_user' and eidfrom == eid: |
806 |
# if for_user was set, delete has already been handled |
|
807 |
break |
|
808 |
else: |
|
809 |
key = session.execute('Any K WHERE P eid %(x)s, P pkey K', |
|
810 |
{'x': eid}, 'x')[0][0] |
|
1398
5fe84a5f7035
rename internal entity types to have CW prefix instead of E
sylvain.thenault@logilab.fr
parents:
1263
diff
changeset
|
811 |
DelCWPropertyOp(session, epropdict=session.vreg.eprop_values, key=key) |
0 | 812 |
|
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
813 |
|
0 | 814 |
def after_add_for_user(session, fromeid, rtype, toeid): |
1398
5fe84a5f7035
rename internal entity types to have CW prefix instead of E
sylvain.thenault@logilab.fr
parents:
1263
diff
changeset
|
815 |
if not session.describe(fromeid)[0] == 'CWProperty': |
0 | 816 |
return |
817 |
key, value = session.execute('Any K,V WHERE P eid %(x)s,P pkey K,P value V', |
|
818 |
{'x': fromeid}, 'x')[0] |
|
819 |
if session.vreg.property_info(key)['sitewide']: |
|
820 |
raise ValidationError(fromeid, |
|
821 |
{'for_user': session._("site-wide property can't be set for user")}) |
|
822 |
for session_ in get_user_sessions(session.repo, toeid): |
|
1398
5fe84a5f7035
rename internal entity types to have CW prefix instead of E
sylvain.thenault@logilab.fr
parents:
1263
diff
changeset
|
823 |
ChangeCWPropertyOp(session, epropdict=session_.user.properties, |
0 | 824 |
key=key, value=value) |
1802
d628defebc17
delete-trailing-whitespace + some copyright update
Adrien Di Mascio <Adrien.DiMascio@logilab.fr>
parents:
1398
diff
changeset
|
825 |
|
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
826 |
|
0 | 827 |
def before_del_for_user(session, fromeid, rtype, toeid): |
828 |
key = session.execute('Any K WHERE P eid %(x)s, P pkey K', |
|
829 |
{'x': fromeid}, 'x')[0][0] |
|
830 |
relation_deleted(session, fromeid, rtype, toeid) |
|
831 |
for session_ in get_user_sessions(session.repo, toeid): |
|
1398
5fe84a5f7035
rename internal entity types to have CW prefix instead of E
sylvain.thenault@logilab.fr
parents:
1263
diff
changeset
|
832 |
DelCWPropertyOp(session, epropdict=session_.user.properties, key=key) |
0 | 833 |
|
2603
e47d63351891
[R hooks] optmize insertion by using direct repository methods when possible
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
2463
diff
changeset
|
834 |
|
0 | 835 |
def _register_eproperty_hooks(hm): |
836 |
"""register workflow related hooks on the hooks manager""" |
|
1398
5fe84a5f7035
rename internal entity types to have CW prefix instead of E
sylvain.thenault@logilab.fr
parents:
1263
diff
changeset
|
837 |
hm.register_hook(after_add_eproperty, 'after_add_entity', 'CWProperty') |
5fe84a5f7035
rename internal entity types to have CW prefix instead of E
sylvain.thenault@logilab.fr
parents:
1263
diff
changeset
|
838 |
hm.register_hook(after_update_eproperty, 'after_update_entity', 'CWProperty') |
5fe84a5f7035
rename internal entity types to have CW prefix instead of E
sylvain.thenault@logilab.fr
parents:
1263
diff
changeset
|
839 |
hm.register_hook(before_del_eproperty, 'before_delete_entity', 'CWProperty') |
0 | 840 |
hm.register_hook(after_add_for_user, 'after_add_relation', 'for_user') |
841 |
hm.register_hook(before_del_for_user, 'before_delete_relation', 'for_user') |