Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
New: TODO il faut incrementer le stock si on supprime une expedition
validee
- Loading branch information
simnandez
committed
Jun 20, 2013
1 parent
b477272
commit ba6b5d2
Showing
5 changed files
with
102 additions
and
48 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ba6b5d2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We must consider that really this is a bug and therefore must be included in 3.4?
ba6b5d2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think we should consider it as a bug. This is the way it works for a long time, and this user case is not usual. I suggest that in the delete confirm box, we add a warning saying that sotck has been decremented after shipment validation and that user must correct it manually after shipment deletion. The real question is how often a user deletes a validated shipment...
ba6b5d2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree with conclusion and explanation of Maxime.
Not blocking and not used frequently so i prefer not include this in 3.4