[CORE-4853] Passing null to strtolower() is deprecated - Blesta

Abdy Franco created issue - 22/Mar/23 6:17 PM

Abdy Franco made changes - 22/Mar/23 6:17 PM

Field	Original Value	New Value
Fix Version/s		5.7.0-b1 [ 11737 ]

Abdy Franco made changes - 28/Mar/23 6:01 PM

Description

When an action is not passed to a controller, internally the action is assumed to be "index" and the index() method is called, however in this case the "$this->action" property has a null value.

This can be found in the following files:
- /app/controllers/admin_main.php
- /app/controllers/client_main.php
- /app/controllers/client_pay.php

Possible solution:
Replace {code:java}strtolower($this->action){code} by {code:java}strtolower($this->action ?? 'index'){code}
_*or*_
Modify the Controller.php class in minPHP Bridge, to set the value of the "action" property to "index", when an action is not passed to the controller.

*Affected Files:*
- /app/controllers/admin_main.php _(Line 35)_
- /app/controllers/client_main.php _(Line 32)_
- /app/controllers/client_pay.php _(Line 19)_
- /app/app_controller.php _(Line 834)_
- /app/client_controller.php _(Line 24)_
- /vendors/minphp/record/src/Record.php _(Line 1501) See PR #10 on phillipsdata/minphp-record_

Possible solution:
Replace {code:java}strtolower($this->action){code} by {code:java}strtolower($this->action ?? 'index'){code}
_*or*_
Modify the Controller.php class in minPHP Bridge, to set the value of the "action" property to "index", when an action is not passed to the controller.

Abdy Franco made changes - 28/Mar/23 6:01 PM

Description

*Affected Files:*
- /app/controllers/admin_main.php _(Line 35)_
- /app/controllers/client_main.php _(Line 32)_
- /app/controllers/client_pay.php _(Line 19)_
- /app/app_controller.php _(Line 834)_
- /app/client_controller.php _(Line 24)_
- /vendors/minphp/record/src/Record.php _(Line 1501) See PR #10 on phillipsdata/minphp-record_

Possible solution:
Replace {code:java}strtolower($this->action){code} by {code:java}strtolower($this->action ?? 'index'){code}
_*or*_
Modify the Controller.php class in minPHP Bridge, to set the value of the "action" property to "index", when an action is not passed to the controller.

*Error:*
{noformat}
strtolower(): Passing null to parameter #1 ($string) of type string is deprecated
{noformat}

*Affected Files:*
- /app/controllers/admin_main.php _(Line 35)_
- /app/controllers/client_main.php _(Line 32)_
- /app/controllers/client_pay.php _(Line 19)_
- /app/app_controller.php _(Line 834)_
- /app/client_controller.php _(Line 24)_
- /vendors/minphp/record/src/Record.php _(Line 1501) See PR #10 on phillipsdata/minphp-record_

Possible solution:
Replace {code:java}strtolower($this->action){code} by {code:java}strtolower($this->action ?? 'index'){code}
_*or*_
Modify the Controller.php class in minPHP Bridge, to set the value of the "action" property to "index", when an action is not passed to the controller.

Abdy Franco made changes - 28/Mar/23 6:20 PM

Description

*Error:*
{noformat}
strtolower(): Passing null to parameter #1 ($string) of type string is deprecated
{noformat}

*Affected Files:*
- /app/controllers/admin_main.php _(Line 35)_
- /app/controllers/client_main.php _(Line 32)_
- /app/controllers/client_pay.php _(Line 19)_
- /app/app_controller.php _(Line 834)_
- /app/client_controller.php _(Line 24)_
- /vendors/minphp/record/src/Record.php _(Line 1501) See PR #10 on phillipsdata/minphp-record_

Possible solution:
Replace {code:java}strtolower($this->action){code} by {code:java}strtolower($this->action ?? 'index'){code}
_*or*_
Modify the Controller.php class in minPHP Bridge, to set the value of the "action" property to "index", when an action is not passed to the controller.

*Error:*
{noformat}
strtolower(): Passing null to parameter #1 ($string) of type string is deprecated
{noformat}

*Affected Files:*
- /app/controllers/admin_main.php _(Line 35)_
- /app/controllers/client_main.php _(Line 32)_
- /app/controllers/client_pay.php _(Line 19)_
- /app/app_controller.php _(Line 834)_
- /app/client_controller.php _(Line 24)_
- /vendors/minphp/record/src/Record.php _(Line 1501) See PR #10 on phillipsdata/minphp-record_

*Possible solution:*
Modify the Controller.php class in minPHP Bridge, to set the value of the "action" property to "index", when an action is not passed to the controller.

Abdy Franco made changes - 05/Apr/23 10:10 PM

Description

*Error:*
{noformat}
strtolower(): Passing null to parameter #1 ($string) of type string is deprecated
{noformat}

*Affected Files:*
- /app/controllers/admin_main.php _(Line 35)_
- /app/controllers/client_main.php _(Line 32)_
- /app/controllers/client_pay.php _(Line 19)_
- /app/app_controller.php _(Line 834)_
- /app/client_controller.php _(Line 24)_
- /vendors/minphp/record/src/Record.php _(Line 1501) See PR #10 on phillipsdata/minphp-record_

*Possible solution:*
Modify the Controller.php class in minPHP Bridge, to set the value of the "action" property to "index", when an action is not passed to the controller.

*Error:*
{noformat}
strtolower(): Passing null to parameter #1 ($string) of type string is deprecated
{noformat}

*Affected Files:*
- /app/controllers/admin_clients.php _(Line 6501)_
- /app/controllers/admin_main.php _(Line 35)_
- /app/controllers/client_main.php _(Line 32)_
- /app/controllers/client_pay.php _(Line 19)_
- /app/app_controller.php _(Line 834)_
- /app/client_controller.php _(Line 24)_
- /vendors/minphp/record/src/Record.php _(Line 1501) See PR #10 on phillipsdata/minphp-record_

*Possible solution:*
Modify the Controller.php class in minPHP Bridge, to set the value of the "action" property to "index", when an action is not passed to the controller.

Abdy Franco made changes - 05/Apr/23 10:49 PM

Description

*Error:*
{noformat}
strtolower(): Passing null to parameter #1 ($string) of type string is deprecated
{noformat}

*Affected Files:*
- /app/controllers/admin_clients.php _(Line 6501)_
- /app/controllers/admin_main.php _(Line 35)_
- /app/controllers/client_main.php _(Line 32)_
- /app/controllers/client_pay.php _(Line 19)_
- /app/app_controller.php _(Line 834)_
- /app/client_controller.php _(Line 24)_
- /vendors/minphp/record/src/Record.php _(Line 1501) See PR #10 on phillipsdata/minphp-record_

*Possible solution:*
Modify the Controller.php class in minPHP Bridge, to set the value of the "action" property to "index", when an action is not passed to the controller.

*Error:*
{noformat}
strtolower(): Passing null to parameter #1 ($string) of type string is deprecated
{noformat}

*Affected Files:*
- /app/controllers/admin_clients.php _(Line 6501)_
- /app/controllers/client_services.php _(Line 1677)_
- /app/controllers/admin_main.php _(Line 35)_
- /app/controllers/client_main.php _(Line 32)_
- /app/controllers/client_pay.php _(Line 19)_
- /app/app_controller.php _(Line 834)_
- /app/client_controller.php _(Line 24)_
- /vendors/minphp/record/src/Record.php _(Line 1501) See PR #10 on phillipsdata/minphp-record_

*Possible solution:*
Modify the Controller.php class in minPHP Bridge, to set the value of the "action" property to "index", when an action is not passed to the controller.

Paul Phillips made changes - 04/May/23 6:23 PM

Fix Version/s		5.7.0 [ 11800 ]
Fix Version/s	5.7.0-b1 [ 11737 ]

Paul Phillips made changes - 24/May/23 4:32 PM

Fix Version/s		5.8.0-b1 [ 11742 ]
Fix Version/s	5.7.0 [ 11800 ]

Jonathan Reissmueller made changes - 26/May/23 3:45 PM

Assignee

Jonathan Reissmueller [ jonathan ]

Jonathan Reissmueller made changes - 26/May/23 5:25 PM

Status	Open [ 1 ]	Closed [ 6 ]
Resolution		Fixed [ 1 ]

Passing null to strtolower() is deprecated

Details

Description

Activity

People

Dates

Agile