Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Hi guys,
I am not sure if this is the correct place to post this so please forgive me if this is an issue that needs to be fixed by Fusion Mobile.
When using Fusion Mobile I am seeing 7 notices appear in the logs for each page view. Below are the 3 unique errors
Notice: Undefined index: sidebar_first in fusion_core_grid_info() (line 392 of sites/all/themes/fusion/fusion_core/template.php).
Notice: Undefined index: sidebar_second in fusion_core_grid_info() (line 392 of /sites/all/themes/fusion/fusion_core/template.php).
Notice: Undefined index: sidebar_second in fusion_core_grid_info() (line 394 of /sites/all/themes/fusion/fusion_core/template.php).
Notice: Undefined index: sidebar_first in fusion_core_preprocess_page() (line 100 of /sites/all/themes/fusion/fusion_core/template.php).
Notice: Undefined index: sidebar_second in fusion_core_preprocess_page() (line 100 of /sites/all/themes/fusion/fusion_core/template.php).
Obviously these functions are referring to regions that don't exist in Fusion Mobile.
Is this a problem with how Fusion Mobile is letting Fusion Core know what regions it has, or is this an issue with Fusion Core assuming that all sub-themes will have these regions?
Comments
Comment #1
aquariumtap CreditAttribution: aquariumtap commented@akoepke - currently sidebar regions are required by Fusion, but I see that those regions are not in use by the fusion mobile theme. Let me find out more information about this...
@stephthegeek - sidebar regions are part of the grid calculations. I could make those regions optional and make any calculated template variables related to sidebar or group widths = 0... what do you think? there are also theme settings that assume those regions are present.
Comment #2
stephthegeek CreditAttribution: stephthegeek commented@aquariumtap: I'm not sure how the technical implementation would work but yeah this is an open question for fusion_mobile. I think it needs to become optional one way or another.
Comment #3
aquariumtap CreditAttribution: aquariumtap commentedokay. I agree. It's come up in other issues as well.
Comment #4
akoepke CreditAttribution: akoepke commentedThanks for working together for a solution guys :)
Comment #5
dddave CreditAttribution: dddave commentedsub
Comment #6
hwasem CreditAttribution: hwasem commentedsubscribe.
Comment #7
thomsol CreditAttribution: thomsol commentedsubscribe
Comment #8
destlund CreditAttribution: destlund commentedsubscribing
Comment #9
WebbehSubscribing, as I am running into this issue as well.
Comment #10
imoreno CreditAttribution: imoreno commentedSubscribing
Comment #11
aquariumtap CreditAttribution: aquariumtap commentedIn my opinion, Fusion isn't the best choice for a theme without sidebars. But if you want to tweak an existing theme, for example, you need to remove all mention of sidebar_first and sidebar_second in the bottom part of the theme's .info file, where "grid adjusted regions", "grid adjusted groups" and "grid adjusted groups fluid" are defined.
If you don't have any sidebars, you can probably just replace this:
With this:
I haven't tested, but that should work.
Comment #12
aquariumtap CreditAttribution: aquariumtap commentedComment #13
dealancer CreditAttribution: dealancer commentedHey guys,
This issue is still alive, using fresh install Fusion + Fusion Mobile, can't fix it, cause fusiom_mobile.info does not contain regions listed in error, so there is nothing to delete.
Comment #14
aquariumtap CreditAttribution: aquariumtap commentedComment #15
Poieo CreditAttribution: Poieo commentedFusion Mobile is no longer necessary with the 7.x-2.x version.