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

Locked fields : php and SQL errors, display regression #1900

Closed
btry opened this issue Aug 10, 2016 · 1 comment
Closed

Locked fields : php and SQL errors, display regression #1900

btry opened this issue Aug 10, 2016 · 1 comment
Assignees
Labels
Milestone

Comments

@btry
Copy link
Contributor

btry commented Aug 10, 2016

Errors logged in GLPi

To reproduce :

Add a lock on manufacturer of a computer and make it different of the actual value reported by the agent. Take into account the rules on manufacturers.

Import the .xml of the computer.

Observed : the tab locks (fields) does no longer show the value in the XML (after rules process), and is the background color is not red.

Expected : no errors in the logs, the value from the .XML should be displayed in the lock (fields) tab.

*** MySQL query error:
SQL: UPDATE `glpi_plugin_fusioninventory_inventorycomputercomputers`
SET `bios_manufacturers_id` = \'\' WHERE `id` =\'12392\'
Error: Incorrect integer value: '' for column 'bios_manufacturers_id' at row 1
Backtrace :
inc/commondbtm.class.php:357
inc/commondbtm.class.php:1119 CommonDBTM->updateInDB()
...nventory/inc/inventorycomputerlib.class.php:173 CommonDBTM->update()
...ry/inc/inventorycomputerinventory.class.php:566 PluginFusioninventoryInventoryComputerLib->updateComputer()
...inventory/inc/inventoryruleimport.class.php:721 PluginFusioninventoryInventoryComputerInventory->rulepassed()
inc/rule.class.php:1404 PluginFusioninventoryInventoryRuleImport->executeActions()
inc/rulecollection.class.php:1459 Rule->process()
...ry/inc/inventorycomputerinventory.class.php:369 RuleCollection->processAllRules()
...ry/inc/inventorycomputerinventory.class.php:108 PluginFusioninventoryInventoryComputerInventory->sendCriteria()
...fusioninventory/inc/communication.class.php:222 PluginFusioninventoryInventoryComputerInventory->import()
...fusioninventory/inc/communication.class.php:455 PluginFusioninventoryCommunication->import()
plugins/fusioninventory/front/communication.php:88 PluginFusioninventoryCommunication->handleOCSCommunication()
plugins/fusioninventory/index.php:51 include_once()

*** PHP Notice(8): Undefined index: manufacturers_id
Backtrace :
...usioninventory/inc/formatconvert.class.php:1866
...usioninventory/inc/formatconvert.class.php:1847 PluginFusioninventoryFormatconvert->replaceids()
...ry/inc/inventorycomputerinventory.class.php:444 PluginFusioninventoryFormatconvert->replaceids()
...inventory/inc/inventoryruleimport.class.php:721 PluginFusioninventoryInventoryComputerInventory->rulepassed()
inc/rule.class.php:1404 PluginFusioninventoryInventoryRuleImport->executeActions()
inc/rulecollection.class.php:1459 Rule->process()
...ry/inc/inventorycomputerinventory.class.php:369 RuleCollection->processAllRules()
...ry/inc/inventorycomputerinventory.class.php:108 PluginFusioninventoryInventoryComputerInventory->sendCriteria()
...fusioninventory/inc/communication.class.php:222 PluginFusioninventoryInventoryComputerInventory->import()
...fusioninventory/inc/communication.class.php:455 PluginFusioninventoryCommunication->import()
plugins/fusioninventory/front/communication.php:88 PluginFusioninventoryCommunication->handleOCSCommunication()
plugins/fusioninventory/index.php:51 include_once()
@btry
Copy link
Contributor Author

btry commented Aug 10, 2016

fix proposal : TECLIB@0d1c3e0

@ddurieux ddurieux added the bug label Aug 10, 2016
@ddurieux ddurieux added this to the 0.90+1.5 milestone Aug 10, 2016
@ddurieux ddurieux self-assigned this Aug 10, 2016
ddurieux pushed a commit to ddurieux/fusioninventory-for-glpi that referenced this issue Aug 11, 2016
ddurieux pushed a commit that referenced this issue Aug 11, 2016
Fix manufacturer problem when lock. closes #1900
ddurieux pushed a commit that referenced this issue Aug 11, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants