PHP IDE :: NuSphere Forums :: View topic

Posted: Tue Jan 07, 2014 9:06 am

Hello i'm working with build 12021. I use phped in a magento project.
I noticed that objects in a foreach loop does not resolve correctly.

Code:

/**
* @var IDPerso_Ads_Model_Mysql4_SubscriptionConfig_Collection
*/
$configs = Mage::getModel('idads/subscriptionconfig')->getCollection();
foreach($configs as $config)
{
/**
* @var IDPerso_Ads_Model_SubscriptionConfig
*/
$config;

//$config is resolved to an ArrayIterator and not to IDPerso_Ads_Model_SubscriptionConfig

}

I already refresh the includes of the project.

Thank you for helping

Posted: Tue Jan 07, 2014 8:03 pm

You probably need to declare the variable before it is assigned, which is before the for loop:

Code:

/**
* @var IDPerso_Ads_Model_Mysql4_SubscriptionConfig_Collection
*/
$configs = Mage::getModel('idads/subscriptionconfig')->getCollection();
/**
* @var $config IDPerso_Ads_Model_SubscriptionConfig
*/
foreach($configs as $config)
{
$config;

}

Like you, I often assign the collection to a variable (easier for debugging), but you can shorten like this if $configs is not used elsewhere:

Code:

/** @var $config IDPerso_Ads_Model_SubscriptionConfig */
foreach(Mage::getModel('idads/subscriptionconfig')->getCollection() as $config)
{
$config;

}

Posted: Wed Jan 08, 2014 5:48 am

Thank you it works.

But the way i coded in my first post worked with older version of phped.

Posted: Wed Jan 08, 2014 6:44 am

If you provide more complete sample illustrating your case, it would be clearer what's up.
I think @var IDPerso_Ads_Model_Mysql4_SubscriptionConfig_Collection would be enough if this is a class implementing iterator with known reference type returned by current() method

As of includes in the project, you do not have to refresh them. Refresh in Code Explorer is always enough. Includes play role only and only if "limit scope to includes" is turned on and we recommend to keep it turned off (meaning do not limit scope to includes)