Tryton - Issues

 

Issue6360

Title Manage readonly state fields on calendar views
Priority feature Status testing
Superseder Nosy List ced, pokoli, resteve, reviewbot, xcodinas
Type behavior Components sao, tryton
Assigned To xcodinas Keywords review
Reviews 31231002, 32181002
View: 31231002, 32181002

Created on 2017-03-14.11:41:23 by resteve, last changed by reviewbot.

Messages
review31231002 updated at https://codereview.tryton.org/31231002/#ps40001
review32181002 updated at https://codereview.tryton.org/32181002/#ps20001
review31231002 updated at https://codereview.tryton.org/31231002/#ps20001
New review32181002 at https://codereview.tryton.org/32181002/#ps1
New review31231002 at https://codereview.tryton.org/31231002/#ps1
msg32613 (view) Author: [hidden] (xcodinas) (Tryton committer) Date: 2017-03-20.14:02:25
I add review31231002 for tryton and review32181002 for sao
msg32498 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2017-03-14.14:26:39
Of course issue4207 is about enforcing the behaviour and the security against such behaviour from the client (or any custom client). So it is not strictly required that why I did not put it as superseder.
msg32497 (view) Author: [hidden] (pokoli) (Tryton committer) Date: 2017-03-14.14:18:55
For me this is a missing feature and not a bug.

I think this can be implemented without implementing issue4207 and I agree with the solution proposed on msg32491 using event_drop for sao.
msg32491 (view) Author: [hidden] (ced) (Tryton committer) (Tryton translator) Date: 2017-03-14.12:02:13
For me, it should mainly be fixed with issue4207.
For the UI, I guess the event should not be updated in on_event_released, the GooCalendar/Fullcalender event should be updated back to original date and the events should be redraw.

PS: None of the older version will be fixed.
msg32490 (view) Author: [hidden] (pokoli) (Tryton committer) Date: 2017-03-14.11:58:37
Indeed the same is possible using sao
msg32488 (view) Author: [hidden] (resteve) Date: 2017-03-14.11:41:23
Scenario:

- Create a production and change to state "assign"
- The Planned Date field in assign state is readonly in form view [1]
- Go to calendar view. You could drag-and-drop the production to other day -> change the planned date [2]

TO fix 3.4, 3.6, 3.8 and 4.2.

[1] https://bitbucket.org/tryton/production/src/923ba8bdea45e99b35c68932cfbced4e4e61951e/production.py?at=default&fileviewer=file-view-default#production.py-43
[2] https://bitbucket.org/tryton/production/src/923ba8bdea45e99b35c68932cfbced4e4e61951e/view/production_calendar.xml?at=default&fileviewer=file-view-default#production_calendar.xml-4
History
Date User Action Args
2017-03-28 10:01:33reviewbotsetmessages: + msg32831
2017-03-27 10:37:35reviewbotsetmessages: + msg32801
2017-03-27 10:37:13reviewbotsetmessages: + msg32799
2017-03-20 14:37:06reviewbotsetmessages: + msg32615
2017-03-20 14:37:05reviewbotsetreviews: 31231002 -> 31231002, 32181002
2017-03-20 14:36:50reviewbotsetnosy: + reviewbot
messages: + msg32614
2017-03-20 14:36:49reviewbotsetreviews: 31231002
keyword: + review
2017-03-20 14:02:26xcodinassetstatus: chatting -> testing
assignedto: xcodinas
messages: + msg32613
nosy: + xcodinas
2017-03-14 14:26:39cedsetmessages: + msg32498
2017-03-14 14:18:55pokolisetpriority: bug -> feature
messages: + msg32497
title: Production could change planned date in calendar view when is readonly -> Manage readonly state fields on calendar views

Showing 10 items. Show all history (warning: this could be VERY long)