Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Inconsistency with propal date and validity date when STATUS_VALIDATED #20304

Closed
bomuux opened this issue Mar 8, 2022 · 1 comment
Closed

Inconsistency with propal date and validity date when STATUS_VALIDATED #20304

bomuux opened this issue Mar 8, 2022 · 1 comment
Assignees
@tiaris
Labels
Bug This is a bug (something does not work as expected)

Comments

@bomuux
Copy link
Contributor

bomuux commented Mar 8, 2022 •
edited
Loading

Bug

Problem concerns this two propal dates :

  • date (function Propal::set_date)
  • validity date (function Propal::set_echeance)

When propal status is STATUS_VALIDATED, this two dates seems to be editable (on propal card) but can't be changed because of queries in corresponding functions : queries require status to be STATUS_DRAFT.

Suggestion : Update should be possible OR edition (pencil) should be removed.

Environment Version

15

Environment OS

No response

Environment Web server

No response

Environment PHP

No response

Environment Database

No response

Environment URL(s)

No response

Expected and actual behavior

No response

Steps to reproduce the behavior

No response

Attached files

No response
@bomuux bomuux added the Bug This is a bug (something does not work as expected) label Mar 8, 2022
@tiaris tiaris self-assigned this Oct 18, 2022
tiaris added a commit to tiaris/dolibarr that referenced this issue Oct 18, 2022
@tiaris
FIX Dolibarr#20304 Inconsistency with propal date and validity date when
123efad 
STATUS_VALIDATED
@tiaris tiaris mentioned this issue Oct 18, 2022
Closed
@tiaris
Copy link
Contributor

tiaris commented Oct 18, 2022

PR added #22618

eldy added a commit that referenced this issue Jun 10, 2023
@eldy
Merge pull request #25040 from tiaris/FIX_20304
fab26b8 
FIX #20304 propaldates update
@eldy eldy closed this as completed in e1939d1 Jun 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tiaris tiaris

Labels
Bug This is a bug (something does not work as expected)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tiaris @bomuux