Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

[BUG]: Creation of dynamic property TestTask::$db is deprecated #16308

Closed
Jeckerson opened this issue Mar 9, 2023 · 3 comments · Fixed by #16376 or #16375
Closed

[BUG]: Creation of dynamic property TestTask::$db is deprecated #16308

Jeckerson opened this issue Mar 9, 2023 · 3 comments · Fixed by #16376 or #16375
Assignees
@Jeckerson
Labels
5.0 The issues we want to solve in the 5.0 release bug A bug report status: medium Medium

Comments

@Jeckerson
Copy link
Member

Describe the bug
Inside task, by calling registered dependency injection service like $this->db will throw ErrorException.

To Reproduce

class TestTask extends Task
{
    public function mainAction(): void
    {
        $this->db->fetchAll('...');
    }
}

Details

  • Phalcon version: 5.2.1
  • PHP Version: 8.2
  • Operating System: any
  • Installation type: PECL
  • Zephir version (if any): 0.17.0
  • Server: Other
@Jeckerson Jeckerson added bug A bug report status: unverified Unverified labels Mar 9, 2023
@Jeckerson Jeckerson self-assigned this Mar 9, 2023
@Jeckerson Jeckerson added 5.0 The issues we want to solve in the 5.0 release and removed status: unverified Unverified labels Mar 9, 2023
@noone-silent
Copy link
Contributor

This bug appears everywhere on php8.2, for example in Class Row too:

PHP Deprecated:  Creation of dynamic property Phalcon\Mvc\Model\Row::$label is deprecated in ...

@joepsyko
Copy link

I have a similar problem

E_DEPRECATED: Creation of dynamic property Phalcon\Mvc\Micro::$request is deprecated {"code":8192,"message":"Creation of dynamic property Phalcon\\Mvc\\Micro::$request is deprecated","file":"/var/www/html....

@niden niden mentioned this issue Jul 17, 2023
Merged
5 tasks
@niden niden added the status: medium Medium label Jul 17, 2023
@niden niden added this to Phalcon v5 Jul 17, 2023
@niden niden moved this to In Progress in Phalcon v5 Jul 17, 2023
@niden niden linked a pull request Jul 17, 2023 that will close this issue
T16308 di dynamic properties 4 #16376
Merged
5 tasks
@niden
Copy link
Member

niden commented Jul 17, 2023

Resolved in #16376

@niden niden closed this as completed Jul 17, 2023
@niden niden moved this from In Progress to Implemented in Phalcon v5 Jul 17, 2023
@niden niden linked a pull request Jul 17, 2023 that will close this issue
Phalcon\Mvc\Model\Row class that extends from stdClass #16375
Merged
5 tasks
@niden niden moved this from Implemented to Released in Phalcon v5 Jul 28, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@Jeckerson Jeckerson

Labels
5.0 The issues we want to solve in the 5.0 release bug A bug report status: medium Medium
Projects
Phalcon v5
Status: Released
Milestone
No milestone
4 participants
@niden @noone-silent @Jeckerson @joepsyko