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.

CommentFileSizeAuthor
#13 omega_kickstart-fix-omega-php-memory-overflow-1814350-13.patch834 bytesjsacksick
#11 omega_kickstart-fix-omega-php-memory-overflow-1814350-11.patch664 bytesjsacksick
#10 omega_kickstart-fix-omega-php-memory-overflow-1814350-10.patch606 bytesjsacksick

Comments

brephraim’s picture

brephraim commented

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?

bojanz’s picture

bojanz commented

That's huge help, thanks!

brephraim’s picture

brephraim commented

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

TimelessDomain’s picture

TimelessDomain commented

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

dinofile’s picture

dinofile commented

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'];

slawless’s picture

slawless commented

This is on the template.php correct?

brephraim’s picture

brephraim commented

dinofile's code change worked for me.

jsacksick’s picture

jsacksick commented
Assigned: Unassigned » cookiz

Did you have the time to try the fix Cookiz ?

jsacksick’s picture

jsacksick commented
Project: Commerce Kickstart » Omega Kickstart
Version: 7.x-2.x-dev » 7.x-3.x-dev
jsacksick’s picture

jsacksick commented
Status: Active » Needs review
StatusFileSize
new omega_kickstart-fix-omega-php-memory-overflow-1814350-10.patch606 bytes

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

jsacksick’s picture

jsacksick commented
StatusFileSize
new omega_kickstart-fix-omega-php-memory-overflow-1814350-11.patch664 bytes

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

jsacksick’s picture

jsacksick commented

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

jsacksick’s picture

jsacksick commented
StatusFileSize
new omega_kickstart-fix-omega-php-memory-overflow-1814350-13.patch834 bytes

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

jsacksick’s picture

jsacksick commented
Status: Needs review » Fixed

Committed to dev

Status: Fixed » Closed (fixed)

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