- 31 May, 2017 15 commits
-
-
Łukasz Nowak authored
-
Łukasz Nowak authored
-
Łukasz Nowak authored
-
Łukasz Nowak authored
-
Łukasz Nowak authored
Since there is constraint on the movement, which requires quantity to match amount of items it is user responsibility to take decision about what to do after using dialogs to fill in items.
-
Łukasz Nowak authored
Quantity on movements shall match amount of items.
-
Łukasz Nowak authored
Movement_isAggregateItemListEnabled allows to define, if given movement is a candidate for aggregation, so extract this part into Movement_isAggregateCandidate.
-
Rafael Monnerat authored
Found implementation with remarks: * test coverage is bootstrapped * destructive approach (abandoning Items) is considered as not implemented * documents and properties follow naming conventions * Delivery_createItemList is reused * support case when amount of items is less than quantity on line * support case when amount if items is more than quantity on line
-
Rafael Monnerat authored
-
Łukasz Nowak authored
-
Łukasz Nowak authored
Since Movement_isQuantityEditable does not say anything about Item related movements, there is no reason to use any condition while updating quantities.
-
Łukasz Nowak authored
It is unsure if quantity should not be editable in case of Items aggregated on the movement. Additionally some designs with Items require setting quantity on movements.
-
Łukasz Nowak authored
Create size_base helper variable, which allows to make test non-sequential.
-
Sebastien Robin authored
For new developments, we should use portal_components. So move theses tests to portal_components before Workflow changes.
-
Sebastien Robin authored
There is no selenium tests yet, but at least we can see the same graph rendered with different gadgets. Later, we will have to find smart ways for testing graphs
-
- 30 May, 2017 5 commits
-
-
Romain Courteaud authored
-
Romain Courteaud authored
[erp5_officejs_appstore_base] configuration_latest_version is set when acception the Software Publication
-
Cédric Le Ninivin authored
-
Romain Courteaud authored
Fix web section templates. Fix appcache for the redirect app.
-
Tristan Cavelier authored
to fit original script behavior, from erp5_web skin folder. Also fixes "KeyError: TraversalRequestNameStack" from `WebPage.getDocumentValue(reference)` on the "View" skin selection.
-
- 29 May, 2017 13 commits
-
-
Sven Franck authored
-
Cédric Le Ninivin authored
-
Romain Courteaud authored
-
Romain Courteaud authored
-
Romain Courteaud authored
-
Romain Courteaud authored
-
Romain Courteaud authored
-
Romain Courteaud authored
-
Romain Courteaud authored
-
Romain Courteaud authored
-
Jérome Perrin authored
To make it easier for users to change their preferences, so that they do not have to create the preference themselves, we tried to pre-create a user preference ready to be configured for each user. It was 59860df3 : an interaction to create a user preference on `Person.setReference` which is more or less the time when this person become a user ( but not really - this was already a weakness of this approach). This calls `Person_createUserPreference` that initialize the preference by introspecting the assignments of the person. This already had a problem that it was working only if the assignments were created before the reference was set on the person. With the new user management introduced in !185 this interaction moved to `Person.setUserId`, which is called in Person's init script. This had the following problems: - All persons have a user id, so all persons have a preference. For sites with many persons that are not actually users, this create useless preferences. - During init, person does not have assignments yet, so `Person_createUserPreference` could not use information from assignment to create preference. The suggested change is to create the preference only when the user click on *Edit my preferences* button. This is done by adding a new `portal_preferences.getActiveUserPreference` method that returns the active user preference and create it if not already existing, this way we do not have to put logic in the user interface scripts. All *Edit my preferences* links should use it like it was done in f62e6651 The `person_interaction_workflow` was completely remove, as the other interaction it was containing - clearCache when deleting the person - was useless . We had to adjust a few tests that was passing thanks to this interaction. /cc @gabriel @vpelletier @kazuhiko @tc /reviewed-on nexedi/erp5!273
-
Jérome Perrin authored
Base.edit has this feature of not actually modifying the properties when the new property value is same as the current one, so when we do `movement.edit(price=x)`, this will cause an implicit getPrice. As price lookup is a bit slow, do not lookup price in this case. /cc @vpelletier @kazuhiko @romain /reviewed-on nexedi/erp5!259
-
Jérome Perrin authored
Fix for [#20160719-1B69F57]( https://nexedi.erp5.net/bug_module/20160719-1B69F57 ) These after clone methods in transformed resources are being too clever and behave very badly when the transformation itself is cloned. The suggested approach is to stop initializing int index & reference when cloning transformed resources ("transformation lines"). This behavior is kept when adding new transformed resource. This become consistent with what we have, for example, with Sale Order Lines in Sale Orders. /cc @luke @seb @gabriel @kazuhiko @romain @Nicolas ( see also nexedi/erp5!148 for more ) /reviewed-on nexedi/erp5!258
-
- 25 May, 2017 7 commits
-
-
Sven Franck authored
-
Sven Franck authored
-
Jérome Perrin authored
Abort transaction, we do not need changes made by test to persist. Execute pending activities before removing persons, the pending activities might be relying on the existence of persons.
-
Jérome Perrin authored
-
Jérome Perrin authored
-
Jérome Perrin authored
-
Jérome Perrin authored
-