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

QUAL add $mysoc->currency and call default lang before setMysoc() #32903

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

hregis
Copy link
Contributor

@hregis hregis commented Feb 1, 2025

No description provided.

@JonBendtsen
Copy link
Contributor

v20.0.2 docker image

[Sat Feb 01 17:51:16.701851 2025] [php:error] [pid 3090:tid 3090] [client ...... PHP Fatal error:  Uncaught Error: Call to undefined function currency_name() in /var/www/html/societe/class/societe.class.php:4407\nStack trace:\n#0 /var/www/html/master.inc.php(226): Societe->setMysoc()\n#1 /var/www/html/main.inc.php(439): require_once('...')\n#2 /var/www/html/product/index.php(33): require('...')\n#3 {main}\n  thrown in /var/www/html/societe/class/societe.class.php on line 4407, referer: ......

@hregis
Copy link
Contributor Author

hregis commented Feb 1, 2025

@JonBendtsen this is for develop branch, add company.lib.php in societe.class.php

Capture d’écran 2025-02-01 à 18 09 20

@JonBendtsen
Copy link
Contributor

JonBendtsen commented Feb 1, 2025

@hregis thanks. Why are the changes to master.inc.php necessary? I get this in my json with only changes to societe.class.php

"currency_code": "DKK",
"currency": "Denmark Krone"

@hregis
Copy link
Contributor Author

hregis commented Feb 1, 2025

@JonBendtsen without the change in master.inc.php the label is the value in database and not the translation

@JonBendtsen
Copy link
Contributor

@JonBendtsen without the change in master.inc.php the label is the value in database and not the translation

ok, I can live with that until it get's included in some v2x docker image

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.

3 participants