I can’t update clients

[elricho]

I use the last version in a new install for php 7.3 version.name = 2019.2.10 version.update_date = 20191015

I put a new client with a name and a email , then I go again to update the client to put another information like telephone adreess and when I submit I get this error in tmp/log/php.log

in the screen said Something went wrong, please try saving the customer again

[25-Nov-2019 08:17:38 America/Los_Angeles] PHP Warning: “continue” targeting switch is equivalent to “break”. Did you mean to use “continue 2”? in /home/rizzi/domains/rizzi.ml/public_html/si/library/HTMLPurifier/HTMLPurifier.standalone.php on line 10943
[25-Nov-2019 08:17:38 America/Los_Angeles] PHP Warning: “continue” targeting switch is equivalent to “break”. Did you mean to use “continue 2”? in /home/rizzi/domains/rizzi.ml/public_html/si/library/HTMLPurifier/HTMLPurifier.standalone.php on line 10949
[25-Nov-2019 08:17:38 America/Los_Angeles] PHP Deprecated: The each() function is deprecated. This message will be suppressed on further calls in /home/rizzi/domains/rizzi.ml/public_html/si/vendor/zendframework/zendframework1/library/Zend/Cache/Backend.php on line 79
[25-Nov-2019 08:18:01 America/Los_Angeles] PHP Warning: “continue” targeting switch is equivalent to “break”. Did you mean to use “continue 2”? in /home/rizzi/domains/rizzi.ml/public_html/si/library/HTMLPurifier/HTMLPurifier.standalone.php on line 10943
[25-Nov-2019 08:18:01 America/Los_Angeles] PHP Warning: “continue” targeting switch is equivalent to “break”. Did you mean to use “continue 2”? in /home/rizzi/domains/rizzi.ml/public_html/si/library/HTMLPurifier/HTMLPurifier.standalone.php on line 10949
[25-Nov-2019 08:18:01 America/Los_Angeles] PHP Deprecated: The each() function is deprecated. This message will be suppressed on further calls in /home/rizzi/domains/rizzi.ml/public_html/si/vendor/zendframework/zendframework1/library/Zend/Cache/Backend.php on line 79
[25-Nov-2019 08:18:01 America/Los_Angeles] PdoDb – debugger(): UPDATE si_customers SET attention = ”, name = ‘Mirta’, department = ”, street_address = ‘Canale’, street_address2 = ”, city = ”, state = ”, zip_code = ”, country = ”, phone = ”, mobile_phone = ”, fax = ”, email = ‘cristianrizzi@gmail.com’, credit_card_holder_name = ”, credit_card_expiry_month = ”, credit_card_expiry_year = ”, notes = ”, default_invoice = ”, custom_field1 = ”, custom_field2 = ”, enabled = 1 WHERE id = 2 AND domain_id = 1
[25-Nov-2019 08:18:01 America/Los_Angeles] PdoDb – query(): Execute error.Array
(
[0] => 22007
[1] => 1366
[2] => Incorrect integer value: ” for column rizzi_si.si_customers.default_invoice at row 1
)

[RRowley]

There is some database setting the is not allowing an empty string to work as the same as the numeric 0 default value for a field. The si_customers table default_invoice field is int type and has a default value of 0. The update command was sending in an empty string (”) when this field was not set. This caused an error to be thrown. This default works on my database but not yours which is why I think it is some DB setting that differs on you set up vs mine.

I updated the customer update screen to send a 0 in for this field. This change has been uploaded to the SI version github. The new SI version is 2019.2.11. You can copy the templates/default/customers/details.tpl file from github and replace the one on your install. See if that helps. If so, you should pull the entire update and replace what you have so the the version and other info gets updated also.

While this might fix the customer update, you might encounter a similar issue on other screens. So determining the database setting that lets mine work and yours not, will be the best all around solution. I’ll keep checking to see if I can find this.

[elricho]

Thank you very much, I will be try with this file today.

Thanks again Rowley!

[Author]

Posts