Hello,

I am testing a new site and using 3.1.Thanks for this great theme which i am a fan of. I've started receiving an error right after installing the Maestro module. the error is :

" Warning: array_shift() expects parameter 1 to be array, string given in adaptivetheme_breadcrumb() (line 87 of /home/example/public_html/dr/sites/all/themes/adaptivetheme/at_core/inc/theme.inc).".

I believe that Maestro module might have triggered something. Any suggesitons?

Best Regards

Comments

bonchak’s picture

Title: Maestro module and adaptivetheme » i can confirm that this relies adaptivetheme

I've just switched to Bartik theme and it works fine with Bartik.

Thanks

bonchak’s picture

Title: i can confirm that this relies adaptivetheme » Maestro module and adaptivetheme

sory for changing the issue title ...

Jeff Burnz’s picture

There must be more to this, in terms of the setup, simply enabling the module did not reproduce the issue. Are there more steps involved? Where do you get this error (what page?).

Jeff Burnz’s picture

Title: Maestro module and adaptivetheme » Maestro module sets $variables['breadcrumb'] to a string in maestro_taskconsole()
Project: AdaptiveTheme » Maestro
Version: 7.x-3.1 » 7.x-1.x-dev
Component: Module Support » Code

I found it in the taskconsole.

Maestro incorrectly sets the value the breadcrumb to an empty string, this is wrong because a theme will expect $variables['breadcrumb'] to be an array in theme_breadcrumb().

I would suggest if the module really does not want to set a breadcrumb to set an empty array, i.e. in maestro_taskconsole():

drupal_set_breadcrumb(array());

This would be preferable to themes needing to first check if $variables['breadcrumb'] is actually an array.

bonchak’s picture

Thank you Jeff for your fast reply,

Sorry for the insufficient info, but i see that you've already regenerated it on task console, which is the same for me. I am new to coding and when changing to Baritk did not reproduce this error i thought this is related to adaptivetheme. So i guess i should adress this issue to maestro.

Thanks for your help and for the great themes...

bonchak’s picture

well Jeff i think you've already done that. Thx

_randy’s picture

Thanks Jeff. I'll pop that change in to the next set of updates I have for Maestro.

blainelang’s picture

Status: Active » Fixed

Thanks @Jeff Burnz, I have committed your fix now.

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

Issue summary: View changes

code tags