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.

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

Workflow for project and task #26920

Open
3 tasks
lmag opened this issue Dec 1, 2023 · 2 comments · May be fixed by #27929
Open
3 tasks

Workflow for project and task #26920

lmag opened this issue Dec 1, 2023 · 2 comments · May be fixed by #27929
Assignees
Labels
Feature request This is a feature request

Comments

@lmag
Copy link
Member

lmag commented Dec 1, 2023

Proposal step to follow by @eldy in (#27929)

  • Step 1 - Manage in php code the both field fk_status and status and set old one as deprecated. So existing modules are compatible and can be updated using the new field name being also compatible.

  • FIX #26920 replace fk_statut by status #27930

  • Step 2 - when we are sure that step 1 us completed (we can use ->oldname and ->newname in code, for both create,update and read, with the same result), then in a next version n+x, we update field in databse with the new name and update sql requests in core code.

  • FIX #26920 use status instead of fk_statut #27929

  • Step 3 - Then in a next version n+y, we remove the duplicated code related to fk_status.

Old proposal by @lmag and FHenry

  • PR1 : Update fkstatut in fkstatus + drop colum (UPDATE llx_project__task SET status=fk_statut; )
  • PR2 : Class task.class.php replace statut by status (FIX #26920 use status instead of fk_statut #27929)
  • PR3 : Replace task fkstatut by status
  • PR4 : DROP COLUMN fk_statut FROM llx_projet task (Via deprecated script Dolibarr v25) (/**
  • PR6 : Take all standard class from module builder
@lmag lmag added the Feature request This is a feature request label Dec 1, 2023
@lmag lmag assigned FHenry and lmag Dec 1, 2023
@atm-maxime
Copy link
Member

We tried something related #20634

@evarisk-micka
Copy link
Member

#27929

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature request This is a feature request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants