Notices

I ended up not having time to apply these from the pastebin last night. Putting them here so they don't get lost. Credit to neclimdul

diff --git advanced_forum.module advanced_forum.module
index 4bae7ee..af7e7c4 100644
--- advanced_forum.module
+++ advanced_forum.module
@@ -185,8 +185,7 @@ function advanced_forum_theme_registry_alter(&$theme_registry) {
$naked_path = drupal_get_path('module', 'advanced_forum') . '/styles/naked';
foreach ($templates as $template) {
// Sanity check in case the template is not being used.
- if (is_array($theme_registry[$template])) {
-
+ if (!empty($theme_registry[$template])) {
// If there was a path in there, store it.
$existing_path = array_shift($theme_registry[$template]['theme paths']);

@@ -380,7 +379,7 @@ function advanced_forum_settings_page() {
function advanced_forum_preprocess_author_pane(&$variables) {
// Author pane is used in various places. The only way to be sure to only
// hit it when we are using it is to check the template suggestion.
- if ($variables['template_suggestion'] == 'advanced_forum-author-pane') {
+ if (!empty($variables['template_suggestion']) && $variables['template_suggestion'] == 'advanced_forum-author-pane') {
// Attempt to override the user picture fetched by Author Pane from core.
$sized_picture = theme('advanced_forum_user_picture', $variables['account']);
if (!empty($sized_picture)) {
@@ -1810,7 +1809,7 @@ function advanced_forum_first_new_comment($nid, $timestamp = 0) {
*/
function advanced_forum_first_new_post_link($node, $comment_count) {
$nid = $node->nid;
- $current_page = $_GET['page'];
+ $current_page = isset($_GET['page']) ? $_GET['page'] : 0;
$number_new_comments = advanced_forum_reply_num_new($nid);

if ($number_new_comments > 0) {
@@ -1827,7 +1826,7 @@ function theme_advanced_forum_topic_pager($pagecount, $topic) {

$topic_pager = advanced_forum_create_topic_pager($pagecount, $topic);

- if (is_array($topic_pager->initial_pages)) {
+ if (isset($topic_pager->initial_pages)) {
return '(' . implode(", ", $topic_pager->initial_pages) . $topic_pager->last_page . ')';
}
}

Fixed, thanks.

Michelle

And the status...

Michelle