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 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 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 commentedokay. I agree. It's come up in other issues as well.
Comment #4
akoepke commentedThanks for working together for a solution guys :)
Comment #5
dddave commentedsub
Comment #6
hwasem commentedsubscribe.
Comment #7
thomsol commentedsubscribe
Comment #8
destlund commentedsubscribing
Comment #9
WebbehSubscribing, as I am running into this issue as well.
Comment #10
imoreno commentedSubscribing
Comment #11
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 commentedComment #13
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 commentedComment #15
Poieo commentedFusion Mobile is no longer necessary with the 7.x-2.x version.