Finish porting Forest Floor to Drupal 7

Same with Drupal 7 beta 1, Call to undefined function, with WSOD. Simply does not work.

Also, BEFORE setting as default theme (which gives WSOD), while still enabled but not "in use", this theme does not show up in configurable settings anywhere other than on its own settings page (which works). The other parts of Drupal does not "see" it at all, it seems.

I have exact same issue.

As do I.

Same problem here.

I have the same problem

Bump - same issue (line 17 for me)
Fatal error: Call to undefined function theme_get_settings() in /home/content/etc/etc/html/Drupal7/sites/all/themes/forest_floor/template.php on line 17

theme_get_settings is not the in the Drupal 7 API reference, so it seems very likely that is no longer a function in Drupal 7. This is why the theme is borking.

Next step: try to figure out a replacement function or etc.

After comparing the theming system of Drupal 7 to the the Drupal 7 version of Forest Floor, it appears that the Forest Floor theme has not been converted to Drupal 7 on even the most basic level.

If you want Forest Floor to work on Drupal 7, you have a lot of converting and hacking to accomplish.

Seems like this is not ported at all, hence it is not a bug, but a task to finish...

This is simply a typo - theme_get_settings should be theme_get_setting.

However, if you do fix the code you will just run up against more issues, including use of functions that are no longer supported in v7.

In other words, this has not really been ported as it is still using obsolete v6 functions, for example node_get_types().

I am looking at this theme myself, and it does look like SOME work has been done to the theme. For example, the CSS is updated to the proper block CSS IDs rather than block-type-#. It might be down to some function calls that simply haven't been ported.

Here's a patch that gets through the two template.php issues I found with this theme.

- there was a on theme_get_setting() function
- function node_type_get_types() is D7 version of node_get_types() in D6

I'm working on a site where the client wants to use a variation of this theme, so if I run into any additional problems I'll check the issue queue and patch accordingly

Please disregard the earlier comment. There were many more incompatibilities to be worked through.

Please find attached a working but not thoroughly tested D7 patch.

Summary of changes:

modified: sites/all/themes/forest_floor/forest_floor.info
.info file ---
removed d6 specific settings / keywords
added default stylsheet: style.css
added regions keys (borrowed heavily from bartik. Hey, go with what works)

modified: sites/all/themes/forest_floor/1024.css
modified: sites/all/themes/forest_floor/style.css
reconfigured style.css, 1024.css ---
removed 1024-specific settings to 1024.css
removed various css settings no longer implemented in d7 version
added a few styles
changed column layout: Beware! This layout may now require 3 columns to look right. I did not check this.

new file: sites/all/themes/forest_floor/800.css
added 800.css ---
moved 800 width layout settings to its own .css file

modified: sites/all/themes/forest_floor/template.php
updated template.php ---
this theme no longer depends on advanced_theme_settings module
800 / 1024 wide css added in process_page()
IE workaround css now added in preprocess_page()
added functionality to visibly hide but include site_name, and slogan if they are toggled "off"

modified: sites/all/themes/forest_floor/page.tpl.php
page.tpl.php ---
borrowed heavily from Bartik, but left many of the original classes / IDs / layout in place so as to not break the theme's original concept
removed right sidebar faded-background styling (just my preference and it was taking me too long to understand it)
moved footer section outside content wrapper
removed THEME BY AVIOSO DESIGNS attribution. (sorry, I did this early on and then committed it. Avioso can put it back)

modified: sites/all/themes/forest_floor/theme-settings.php
theme-settings.php ---
migrated to a drupal standard settings form: 800 / 1024 toggle is now a pair of radio buttons

A few more bug fixes and enhancements.
Summary:

commit 36ec8b8122e4ddd792bd929578c0f47234a09bcb
Author: Mark Ewers
Date: Mon Mar 5 13:47:31 2012 -0600

main menu styling

modified: sites/all/themes/forest_floor/style.css
main menu is now properly styled (loyal to the original theme)

commit 57058f175e59128825a5f9d9b86083c4057a8de7
Author: Mark Ewers
Date: Mon Mar 5 13:46:37 2012 -0600

added code for above_header region

modified: sites/all/themes/forest_floor/page.tpl.php

commit b26911d07f345c3850baa97ce811e0d37f56cfbf
Author: Mark Ewers
Date: Mon Mar 5 13:44:43 2012 -0600

Added a new region above the header graphic

modified: sites/all/themes/forest_floor/forest_floor.info
new region: above_header ... Above the header

commit 590e6bc6a48991f37f106e7625cb5bdadfcc4ad3
Author: Mark Ewers
Date: Mon Mar 5 13:43:30 2012 -0600

bug fix: variable names!!

$vars != $variables