Download & Extend
Omega Kickstart » Issues

Omega PHP memory overflow

Posted by brephraim on October 16, 2012 at 3:14pm
Project:Omega Kickstart
Version:7.x-3.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Cookiz
Status:closed (fixed)

Issue Summary

I upgraded today from rc1 to -dev, and as a result started getting errors like this:

Fatal error: Allowed memory size of 134217728 bytes exhausted (tried to allocate 9020 bytes) in .../public_html/profiles/commerce_kickstart/themes/omega_kickstart/template.php on line 35

So I upped the PHP memory limit to 512M, and corresponding errors occurred. So I upped it to 1 gig, the error went away, but I could no longer even reach the page, I got a 'connection reset' page instead.

In the meanwhile, admin theme pages continued to work.

So I downgraded to the rc1 version of all the themes, and the problems went away.

Just thought I'd let someone know.

Login or register to post comments

Comments

#1

Posted by brephraim on October 24, 2012 at 2:04pm

The error is coming from this code:

function omega_kickstart_css_alter(&$css) {
  foreach($css as $key => $item) {
    if (isset($item['basename']) && LANGUAGE_RTL) {
      $css[$item['basename']]['basename'] = 'RTL::' . $item['basename'];
    }
  }
}

For the time being, I have the latest profiles/commerce_kickstart/themes folder from rc2, commented out the above function, and everything appears to be working.

Any help?

#2

Posted by bojanz on October 24, 2012 at 2:58pm

That's huge help, thanks!

#3

Posted by brephraim on October 24, 2012 at 3:29pm

Yeah, that was basically a poorly disguised "bump"...

#4

Posted by TimelessDomain on November 6, 2012 at 1:16pm

Related - #1828058: Undefined Index notices and some warnings with fonts.com, omega sub-theme, commerce kickstart 2 profile.

#5

Posted by dinofile on November 26, 2012 at 4:33am

I am still getting the problem in Omega Kickstart 7.x-3.0-rc2 ...

don't know if its correct, but to stop memory crash problem, in that function I commented out this line and added the line below it (basically used $key instead of $item['basename']). Seems to work, but don't really know what is going on :)

//$css[$item['basename']]['basename'] = 'RTL::' . $item['basename'];

$css[$key]['basename'] = 'RTL::' . $item['basename'];

#6

Posted by slawless on December 24, 2012 at 8:53am

This is on the template.php correct?

#7

Posted by brephraim on January 2, 2013 at 8:39pm

dinofile's code change worked for me.

#8

Posted by jsacksick on January 3, 2013 at 2:56pm
Assigned to:Anonymous» Cookiz

Did you have the time to try the fix Cookiz ?

#9

Posted by jsacksick on January 17, 2013 at 1:02pm
Project:Commerce Kickstart» Omega Kickstart
Version:7.x-2.x-dev» 7.x-3.x-dev

#10

Posted by jsacksick on January 17, 2013 at 1:05pm
Status:active» needs review

Here's the patch that also update the LANGUAGE_RTL condition, only alter the css if the language direction is RTL.

AttachmentSize
omega_kickstart-fix-omega-php-memory-overflow-1814350-10.patch 606 bytes

#11

Posted by jsacksick on January 17, 2013 at 1:20pm

Same patch with the $language->direction check before the loop

AttachmentSize
omega_kickstart-fix-omega-php-memory-overflow-1814350-11.patch 664 bytes

#12

Posted by jsacksick on January 17, 2013 at 4:27pm

Just FYI, this fix has been added to Kickstart v2.1

#13

Posted by jsacksick on February 12, 2013 at 2:28pm

Ok the last patch introduced a regression in KS, and wasn't working at all, here's a new patch.

AttachmentSize
omega_kickstart-fix-omega-php-memory-overflow-1814350-13.patch 834 bytes

#14

Posted by jsacksick on February 13, 2013 at 3:53pm
Status:needs review» fixed

Committed to dev

#15

Posted by System Message on February 27, 2013 at 4:00pm
Status:fixed» closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

nobody click here