Closed (fixed)
Project:
Maestro
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
29 Dec 2012 at 23:26 UTC
Updated:
27 May 2014 at 19:38 UTC
Jump to comment: Most recent
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
Comment #1
bonchak commentedI've just switched to Bartik theme and it works fine with Bartik.
Thanks
Comment #2
bonchak commentedsory for changing the issue title ...
Comment #3
Jeff Burnz commentedThere 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?).
Comment #4
Jeff Burnz commentedI 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 intheme_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():This would be preferable to themes needing to first check if
$variables['breadcrumb']is actually an array.Comment #5
bonchak commentedThank 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...
Comment #6
bonchak commentedwell Jeff i think you've already done that. Thx
Comment #7
_randy commentedThanks Jeff. I'll pop that change in to the next set of updates I have for Maestro.
Comment #8
blainelang commentedThanks @Jeff Burnz, I have committed your fix now.
Comment #9.0
(not verified) commentedcode tags