Skip to content
This repository has been archived by the owner on Oct 4, 2024. It is now read-only.

"Woosync total delta" calculations causes SQL error in invoice list #115

Open
marc-dll opened this issue Nov 9, 2023 · 0 comments
Open

Comments

@marc-dll
Copy link
Contributor

marc-dll commented Nov 9, 2023

At least up to Dolibarr v16, the calculation of the column "Woosync total delta" in invoice list causes an SQL error.

Up to this version, Dolibarr builds the SQL query to count objects in the list by replacing info the query that fetches the results.

This replacement is peculiar, and the way it is done doesn't allow the + sign to be there... which your module does :/

See https://github.com/Dolibarr/dolibarr/blob/16.0/htdocs/compta/facture/list.php#L871 to see what I mean.

I didn't try in Dolibarr v17+, as the way this is done is different, I guess this works there.

I created a PR in the core to try to fix the underlying issue : Dolibarr/dolibarr#26525. In the eventuality that it is merged, you have nothing to do ! But else, there is a workaround to be found.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant