Drupal 7 upgrade [#1121678]

I am currently getting these errors:

Notice: Undefined variable: main_group_width in include() (line 57 of /sw/var/www/d7-dev/sites/all/themes/abstract/templates/page.tpl.php).
Notice: Undefined index: sidebar_first in fusion_core_grid_info() (line 342 of /sw/var/www/d7-dev/sites/all/themes/fusion/fusion_core/template.php).
Notice: Undefined index: sidebar_second in fusion_core_grid_info() (line 344 of /sw/var/www/d7-dev/sites/all/themes/fusion/fusion_core/template.php).
Notice: Undefined index: sidebar_first in fusion_core_grid_info() (line 342 of /sw/var/www/d7-dev/sites/all/themes/fusion/fusion_core/template.php).
Notice: Undefined index: sidebar_second in fusion_core_grid_info() (line 342 of /sw/var/www/d7-dev/sites/all/themes/fusion/fusion_core/template.php).
Notice: Undefined index: sidebar_first in fusion_core_preprocess_page() (line 79 of /sw/var/www/d7-dev/sites/all/themes/fusion/fusion_core/template.php).
Notice: Undefined index: sidebar_second in fusion_core_preprocess_page() (line 79 of /sw/var/www/d7-dev/sites/all/themes/fusion/fusion_core/template.php).

Right now the content and right sidebars have full width. Fusion its still in dev snapshot and there is no usable Skinr for D7, so this issue might have to wait a little.

Comment	File	Size	Author
#11	abstract-drupal-7-upgrade-1121678-9.patch	1.56 KB	mmenavas
#5	0001-Issue-1121678-by-ipwa-mmenavas-sidebar-widths-Update.patch	8.18 KB	mmenavas

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

sbsimmons86 CreditAttribution: sbsimmons86 commented 12 April 2011 at 21:02

Subscribing...Im having the same issue.

Comment #2

ipwa CreditAttribution: ipwa commented 12 April 2011 at 21:50

Are you getting all these errors??

With the latest dev snapshot you should only be getting:

Notice: Undefined variable: main_group_width in include() (line 57 of /sw/var/www/d7-dev/sites/all/themes/abstract/templates/page.tpl.php).

Make sure you clear your cache.

Comment #3

sbsimmons86 CreditAttribution: sbsimmons86 commented 13 April 2011 at 02:35

Yeah, Im only getting the one error now.

Comment #4

ipwa CreditAttribution: ipwa commented 13 April 2011 at 04:35

Awesome thanks. As I said before Fusion is still dev snapshot, so I'll try to fix this bug, but there's no rush until there's at least a fusion beta.

Comment #5

mmenavas CreditAttribution: mmenavas commented 22 May 2011 at 01:48

File	Size
0001-Issue-1121678-by-ipwa-mmenavas-sidebar-widths-Update.patch	8.18 KB

This patch does not solve this issue entirely, but it corrects the page.tpl.php file to render all regions (except for 'Content to') correctly. Stylesheets and div classes might have to be updated

Comment #6

ipwa CreditAttribution: ipwa commented 23 May 2011 at 03:05

Committed: http://drupal.org/commitlog/commit/12322/33f7151a27a5bec5845d7778dc31a95...

Thanks :)

Comment #7

ipwa CreditAttribution: ipwa commented 23 May 2011 at 03:06

Status:

Active

» Needs work

Comment #8

ipwa CreditAttribution: ipwa commented 23 May 2011 at 03:06

Title:

sidebar widths (Fusion 7.x)

» Drupal 7 upgrade

Comment #9

ipwa CreditAttribution: ipwa commented 23 May 2011 at 04:23

Just committed: http://drupal.org/commitlog/commit/12322/732bf82947171dc01ed90f743bd4bbf...

Now we have unstable1 for Drupal 7: http://drupal.org/node/1165662

Didn't want to do an alpha just yet because there are a couple of things to sort out like the 0 that comes from the node.tpl.php probably.

Main menus don't work in Fusion like regular themes btw, you need to assign your main menu block to the main menu region.

@mmenavas if you're going to continue helping out with patches, remember to git pull so you get my latest changes locally, thanks so much for the help.

Comment #10

mmenavas CreditAttribution: mmenavas commented 25 May 2011 at 19:01

No problem! I'm glad to help. I'll pick this up again in future code sprints.

Comment #11

mmenavas CreditAttribution: mmenavas commented 23 June 2011 at 04:14

File	Size
abstract-drupal-7-upgrade-1121678-9.patch	1.56 KB

This patch cleans up part of node.tpl.php. I think abstract.info and page.tpl.php need to be updated to remove one of the sidebars as in Abstract 6.x-1x.