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

FIX #20279 Accountancy - PostGreSQL - Error on mass update lines already binded FPC21+ #20297

Merged
merged 1 commit into from
Mar 24, 2022

Conversation

aspangaro
Copy link
Member

@aspangaro aspangaro commented Mar 8, 2022

PostgreSQL Documentation UPDATE

column_name

The name of a column in the table named by table_name. The column name can be qualified with a subfield name or array subscript, if needed. Do not include the table's name in the specification of a target column — for example, UPDATE tab SET tab.col = 1 is invalid.

Copy link

@soundheritage soundheritage left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello,
I just do the modification on my Dolibarr V15 of test, with my postgreySQL DataBase, and it's work perfectly.
I do the same modification on Dolibarr V12 production, it's ok too.

So thanks for the finding. This fix will be implemented on V15 version or the next ?
You can close my ticket
Thank you for your reactivity on this topic !

If you can take a look also for this old ticket regarding accounting currency problems, more critical issue and probaly more complex : #17462 (comment)

@aspangaro
Copy link
Member Author

Thanks for your returns

So thanks for the finding. This fix will be implemented on V15 version or the next ?

When the implementation of this patch in v12 has been validated, higher versions will be updated

@eldy eldy merged commit 76470c9 into Dolibarr:12.0 Mar 24, 2022
@aspangaro aspangaro changed the title FIX #20279 Accountancy - PostGreSQL - Error on mass update lines already binded FIX #20279 Accountancy - PostGreSQL - Error on mass update lines already binded FPC21+ Nov 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants