FrontAccounting forum — PHP to 8.2 errors with FA 2.4.17

Re: PHP to 8.2 errors with FA 2.4.17

null@example.com (seahawk) — Wed, 17 Apr 2024 08:02:29 +0000

trafficpest wrote:

So its working out now? It is a pretty old plugin, and looks like php has changed to be quite a bit more strict recently.
downgrade php or upgrade FA are best bets

I cannot downgrade on PHP since I am running Joomla sites where I need the higher PHP. @Joe solutions above worked, but when I upgraded to 2.4.18 the errors above reappeared, so I just used my replacement files and then we were up and running again.

Re: PHP to 8.2 errors with FA 2.4.17

null@example.com (trafficpest) — Tue, 16 Apr 2024 20:36:35 +0000

So its working out now? It is a pretty old plugin, and looks like php has changed to be quite a bit more strict recently.
downgrade php or upgrade FA are best bets

Re: PHP to 8.2 errors with FA 2.4.17

null@example.com (seahawk) — Tue, 16 Apr 2024 08:15:55 +0000

Were these changes implemented in 2.4.18, since after the update I had to make the changes again on the same files since the errors reappeared?

Re: PHP to 8.2 errors with FA 2.4.17

null@example.com (seahawk) — Tue, 02 Apr 2024 07:41:01 +0000

Thanks Joe.

Re: PHP to 8.2 errors with FA 2.4.17

null@example.com (joe) — Mon, 01 Apr 2024 08:09:08 +0000

@seahawk
This is an old issue for Windows installations. We are working on that. In the meantime please set the following variabel in config.php to 1:

$no_check_edit_conflicts = 1;

Joe
PS. This is NOT a php8 bug. This issue range over several php versions.

Re: PHP to 8.2 errors with FA 2.4.17

null@example.com (seahawk) — Mon, 11 Mar 2024 08:39:12 +0000

Get the following message after the above updates:

"This edit session has been abandoned by opening sales document in another browser tab. You cannot edit more than one sales document at once."

This happened after I loaded a new customer and wanted to issue a direct invoice. I was still in the same tab. Using Edge.

Re: PHP to 8.2 errors with FA 2.4.17

null@example.com (kvvaradha) — Mon, 26 Feb 2024 12:35:16 +0000

Update your FA to make it work.

Re: PHP to 8.2 errors with FA 2.4.17

null@example.com (arunkumartomail) — Fri, 23 Feb 2024 19:31:46 +0000

[23-Feb-2024 19:29:00 UTC] 0:stalwartshopping:gettext.inc:74:[before upgrade] strtoupper(): Passing null to parameter #1 ($string) of type string is deprecated
[23-Feb-2024 19:29:00 UTC] 0:stalwartshopping:gettext.inc:75:[before upgrade] strtolower(): Passing null to parameter #1 ($string) of type string is deprecated
[23-Feb-2024 19:29:00 UTC] 0:stalwartshopping:JsHttpRequest.php:166:[before upgrade] preg_match(): Passing null to parameter #2 ($subject) of type string is deprecated
[23-Feb-2024 19:29:00 UTC] 0:stalwartshopping:JsHttpRequest.php:167:[before upgrade] strtolower(): Passing null to parameter #1 ($string) of type string is deprecated
[23-Feb-2024 19:29:00 UTC] 0:stalwartshopping:connect_db_mysqli.inc:282: Trying to access array offset on value of type null
[23-Feb-2024 19:29:00 UTC] 0:stalwartshopping:connect_db_mysqli.inc:282: strtoupper(): Passing null to parameter #1 ($string) of type string is deprecated
I\m getting error

Re: PHP to 8.2 errors with FA 2.4.17

null@example.com (seahawk) — Fri, 23 Feb 2024 18:48:58 +0000

On the setup Tab/ Company Setup gives the following error when adding the company logo and selecting update:

Creation of dynamic property TCPDF::$padding is deprecated in file: /home/mysite/public_html/accounts/reporting/includes/tcpdf.php at line 1172

Re: PHP to 8.2 errors with FA 2.4.17

null@example.com (seahawk) — Fri, 23 Feb 2024 18:40:56 +0000

joe wrote:

I understand that you guys are very busy.
Anyhow, I am sure the fixes are ok and I am pushing the files to stable repo now.
Joe

Thank you for the changes. I have updated it as indicated by you and it works, the errors have gone.

Sorry for not coming back earlier, I did not receive the notification.

I appreciate your assistance. I trust that this will be correct in future updates.

Re: PHP to 8.2 errors with FA 2.4.17

null@example.com (apmuthu) — Fri, 23 Feb 2024 16:44:44 +0000

Screenshot missing.
Check the Post release notes for FA 2.4.16 and later:
https://frontaccounting.com/punbb/viewtopic.php?id=10282
Check CHANGELOG for your version:

2023-10-10 Joe Hunt
Release 2.4.17
M    version.php

2023-09-22 Joe Hunt
Bug Found in Supplier Invoice in Calculating Unit Tax
M    purchasing/includes/db/invoice_db.inc

2023-09-21 Joe Hunt
New empty language empty.po file
M    lang/new_language_template/LC_MESSAGES/empty.po

2023-08-07/08 Joe Hunt
PHP 8.2 fixes - 4 commits
M    admin\includes\fa_patch.class.inc
M    includes\hooks.inc
M    includes\prefs\userprefs.inc
M    includes\ui\allocation_cart.inc
M    includes\ui\attachment.inc
M    includes\ui\contacts_view.inc
M    purchasing\inquiry\supplier_inquiry.php
M    reporting\rep110.php
M    reporting\includes\class.graphic.inc
M    sales\includes\cart_class.inc
M    sales\inquiry\customer_inquiry.php

2023-08-01/06 Joe Hunt
Added Payment Link (Payment Service) for XUMM. A Dutch free Payment Service and re-run
M    includes/ui/ui_view.inc

2023-05-03 / 2023-07-03 Joe Hunt
Mark Inactive Customers and Suppliers in Reports (when no Filter). Bug 5709: Calculation fixed.
M    reporting/rep101.php
M    reporting/rep102.php
M    reporting/rep115.php
M    reporting/rep201.php
M    reporting/rep202.php
M    reporting/rep203.php
M    reporting/rep206.php

2023-05-03 Joe Hunt
Release 2.4.16
M    lang/new_language_template/LC_MESSAGES/empty.po
M    version.php

Re: PHP to 8.2 errors with FA 2.4.17

null@example.com (arunkumartomail) — Fri, 23 Feb 2024 16:25:13 +0000

Please note that FrontAccounting 2.4.16 unknown error in the server based. Could anyone can provide me solution. Screen shot attached here.
There is no error only it shows error symbol.

Re: PHP to 8.2 errors with FA 2.4.17

null@example.com (joe) — Thu, 18 Jan 2024 09:11:03 +0000

I understand that you guys are very busy.

Anyhow, I am sure the fixes are ok and I am pushing the files to stable repo now.

Joe

Re: PHP to 8.2 errors with FA 2.4.17

null@example.com (joe) — Tue, 16 Jan 2024 15:03:53 +0000

@seahawk and @kvvaradha

Hello again,

I have now fixed the Workbook.php1 and the exel_report.inc files.

Please download them, rename Workbook.php1 to Workbook.php and replace on your test server. Tell me if all the deprecation errors have vanished. (They should).

If this is also ok, then we can commit the files to stable repo.

Re: PHP to 8.2 errors with FA 2.4.17

null@example.com (joe) — Tue, 16 Jan 2024 13:36:33 +0000

@seahawk

Regarding first error line 66 in frontaccounting.php: Creation of dynamic property front_accounting::$renderer is deprecated in file.

Please insert the following line 31 in frontaccounting.php:

var $renderer = null;

It should remove the 8.2 error, right? Please tell me.

Regarding second error line 267 in sales_order_entry.php. Please change the line to:

$cart->reference = get_post('ref');

Please tell me if it helps, thanks.

Regarding the third error, I need some time to fix the workbook.php file. This is not our own, so I have to fixi the file.
Will be back on this.

Joe