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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

brephraim’s picture

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

That's huge help, thanks!

brephraim’s picture

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

TimelessDomain’s picture

dinofile’s picture

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

This is on the template.php correct?

brephraim’s picture

dinofile's code change worked for me.

jsacksick’s picture

Assigned: Unassigned » Cookiz

Did you have the time to try the fix Cookiz ?

jsacksick’s picture

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

Status: Active » Needs review
FileSize
606 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

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

jsacksick’s picture

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

jsacksick’s picture

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

jsacksick’s picture

Status: Needs review » Fixed

Committed to dev

Status: Fixed » Closed (fixed)

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