Fix inventory per org intallation#1999
Conversation
…n on update and when installing demo data
|
What's about existing installations? Are they correct because the first org_id will be 1 and that fits the ID of the field that should be updated? |
|
I missed that... |
…ganization id instead of the corresponding field id
|
@Fasse: I have now implemented an additional update step to resolve incorrectly assigned selection options that occurred during new installations and when creating new organisations. |
| (ifo_inf_id, ifo_value, ifo_system, ifo_sequence) | ||
| VALUES (?, ?, ?, ?)'; | ||
| $this->db->queryPrepared($sql, array($statusFieldId, $statusOption['inf_name'], true, $statusOption['ifo_sequence'])); |
There was a problem hiding this comment.
You should set the ifo_system = true in the SQL Statement. Since all parameters are handled as Strings per default, PostgreSQL will not accept true as a parameter.
After that change you can merge this.
|
I add the Postgres compatiblity to this branch. @MightyMCoder Can you merge this PR? |
Fix inventory per org intallation
2 participants
closes #1997 #1995