Using modelsManager in model

Question

Using modelsManager in model

Ahmet Bora Sep '16

Created Sep '16	Last Reply Sep '16	Replies 5	Views 1601	Votes 0

Ahmet Bora 58.1k

Sep '16

Hi,

I'm trying modelsManager in model and getting following error:

( ! ) Fatal error: Cannot redeclare class Apphalcon\Models\Portfolios in D:\wamp\www\webrati\app\models\Portfolios.php on line 178

Here my custom method in Portfolios model.

 public static function getRecents()
 {
    $modelsManager = Di::getDefault()->getModelsManager();

    $phql = "   SELECT      p.*
                FROM        portfolios as p
                WHERE       u.is_active = :active:                  
                ORDER BY    p.created DESC";

    $query = $modelsManager->createQuery($phql);
    $results = $query->execute(
        [
            'active' => '1'
        ]
    );

    return $results;    
 }

What is missing? Thanks

Wojciech Ślawski
145.0k

Accepted
answer

Sep '16

As you see you registering models dir, events dir and library dir both times. Just remove this:

$loader->registerDirs(
    array(
        $config->application->eventsDir,
        $config->application->modelsDir,
        $config->application->libraryDir
    )
);

Edwin Manuel Cerrón Angeles · Answer 1 · 2016-09-01T08:13:16-07:00

Edwin Manuel Cerrón Angeles
40.1k

Sep '16

Paste you loader.php

Are you using namespaces?

Ahmet Bora · Answer 2 · 2016-09-01T08:16:30-07:00

Yes, i'm using namespaces:

namespace Apphalcon\Models;

Here my loader:

<?php

use Phalcon\Loader;

$loader = new Loader();

/**
 * We're a registering a set of directories taken from the configuration file
 */
$loader->registerDirs(
    array(
        $config->application->eventsDir,
        $config->application->modelsDir,
        $config->application->libraryDir
    )
);

$loader->registerNamespaces([
    'Apphalcon\Events'          => $config->application->eventsDir,
    'Apphalcon\Tasks'           => $config->application->tasksDir,
    'Apphalcon\Models'          => $config->application->modelsDir,
    'Apphalcon\Controllers'     => $config->application->controllersDir,
    'Apphalcon\Forms'           => $config->application->formsDir,
    'Apphalcon'                 => $config->application->libraryDir
]);

$loader->register();

// Use composer autoloader to load vendor classes
require_once __DIR__ . '/../../vendor/autoload.php';

Lajos Bencz · Answer 3 · 2016-09-01T08:20:02-07:00

It seems like you're loding the model file a second time somehow, I don't think it has to do anything with the DI...

Edwin Manuel Cerrón Angeles · Answer 4 · 2016-09-01T08:24:36-07:00

Call Portofolios:

$foo = new Apphalcon\Models\Portfolios();

Error is because: It means you've already created a class.

For instance:

class Foo {}

// some code here

class Foo {}

That second Foo would throw the error.

Ahmet Bora · Answer 5 · 2016-09-01T08:26:38-07:00

As you see you registering models dir, events dir and library dir both times. Just remove this:
$loader->registerDirs(
   array(
       $config->application->eventsDir,
       $config->application->modelsDir,
       $config->application->libraryDir
   )
);

When these line removed, it works. But it was working before using modelsManager. Is this normal?

Lajos Bencz · Answer 6 · 2016-09-02T01:48:53-07:00

Lajos Bencz
77.7k

Sep '16

Yes it's normal, thanks to lazy-loading