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

fk_typent et fk_user_creat inverted into Societe::create() #21140

Closed
pascaldaugeron opened this issue Jun 7, 2022 · 0 comments · Fixed by #21167
Closed

fk_typent et fk_user_creat inverted into Societe::create() #21140

pascaldaugeron opened this issue Jun 7, 2022 · 0 comments · Fixed by #21167
Labels
Bug This is a bug (something does not work as expected)

Comments

@pascaldaugeron
Copy link

pascaldaugeron commented Jun 7, 2022

Bug

Hi

See into 14.0.5, and develop.

file /societe/class/societe.class.php

https://github.com/Dolibarr/dolibarr/blob/develop/htdocs/societe/class/societe.class.php

lines 929 et 930 :

$sql .= ", fk_user_creat";
$sql .= ", fk_typent";

lines 945 et 946 :

$sql .= ", ".(!empty($this->typent_id) ? ((int) $this->typent_id) : "null");
$sql .= ", ".(!empty($user->id) ? ((int) $user->id) : "null");

fk_user_creat and fk_typent are inverted between definition and VALUES

Pascal

Environment Version

No response

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

@pascaldaugeron pascaldaugeron added the Bug This is a bug (something does not work as expected) label Jun 7, 2022
@pascaldaugeron pascaldaugeron changed the title fk_typent et fk_user_creat inversés dans Societe::create() fk_typent et fk_user_creat inverted into Societe::create() Jun 8, 2022
badre-agtaib pushed a commit to badre-agtaib/dolibarr that referenced this issue Jun 8, 2022
eldy added a commit that referenced this issue Jun 9, 2022
eldy pushed a commit that referenced this issue Jun 9, 2022
Conflicts:
	htdocs/societe/class/societe.class.php
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug This is a bug (something does not work as expected)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant