Advanced Forum and Advanced Profile Kit integration

Posted by Michelle on October 28, 2008 at 3:59am

4 followers

Issue Summary

AF (and soon APK) allows other modules to add variables to the "author pane" via hook. I initially took the easy route by simply writing the integration myself and included the code in AF/APK. As I get more and more requests to integrate module X, I realized this was a bad approach and am starting the process to get the hooks into the individual contrib modules.

All I need from you is to either drop the attached files into your module directories (remove the .txt that uploading requires) or take the functions out of them and put it in your module files. Both methods work, just choose the one that works best from you. If you aren't willing to do either, please let me know.

For more information, see Adding variables to the author pane

Thanks,

Michelle

Attachment	Size
user_stats.author-pane.inc5_.txt	619 bytes
user_stats.author-pane.inc6_.txt	619 bytes

Comments

#1

Posted by Liam McDermott on November 16, 2008 at 5:40am

Status:

active

» fixed

This is now in CVS (for both five and six branches). I haven't tested it at all though. :)

#2

Posted by Michelle on November 16, 2008 at 5:21am

Yay! Thanks. :) It should work fine. The code has been in AF for a while. I just took the file and copied it to the issue. I'll give it whirl tomorrow. Did you put it in both branches?

[Edit: Dumb question... I just looked at the commit log. LOL! Thanks again!]

Michelle

#3

Posted by Liam McDermott on November 16, 2008 at 5:41am

lol, I just edited my original post, thinking I was stupid for not letting you know the code is in both branches. :D

#4

Posted by Michelle on November 16, 2008 at 5:42am

Yay for crossposting :)

Thanks and goodnight.

Michelle

#5

Posted by System Message on November 30, 2008 at 5:46am

Status:

fixed

» closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.

#6

Posted by Manamarak on January 12, 2009 at 3:31am

Category:	feature request	» bug report
Status:	closed (fixed)	» active

Integration works fine, until, that is, I clear the site cache.

I then come up against the following message:

"Fatal error: Cannot redeclare user_stats_preprocess_author_pane() (previously declared in D:\websites\illustratedphotography\test-site\modules\user_stats\user_stats.module:704) in D:\websites\illustratedphotography\test-site\modules\author_pane\modules\user_stats.author-pane.inc on line 24"

All page refreshes then disappears until you find a way of switching off the User Stats module, at which point everything returns to normal.

Have repeated the process several times, with the exact same effect.

Be warned, have a tab open with /admin/build/modules, or else there is no way of switching off the module.

I am using:

Latest Drupal ver.
User Stats 6.x-1.x-dev
Advanced Forum 6.x-1.0-unstable1
Author Pane 6.x-1.0-unstable1

#7

Posted by Liam McDermott on January 12, 2009 at 6:07am

Status:

active

» closed (won't fix)

Unfortunately this is a completely seperate issue and needs to be in its own bug report. Every bug must have its own report, or things become confusing. :)

Also, I double-checked the code in User Stats and the function is only declared once. So I'm going to assume the problem is caused by Advanced Forum (or maybe even Drupal) including the user_stats.module file twice.

The information you provided is good though: please copy and paste that over to a new issue on the Advanced Forum list. Bonus points if you come back here and note the URL of the new issue.

Thanks. :)

#8

Posted by Michelle on January 12, 2009 at 1:31pm

@Liam: No worries. This hijacking is a dupe, anyway. It's already fixed in the dev of AP. I accidentally included my own version in the D6 version because, for some reason, I thought this had only been committed here to 5.x.

@Manamarak: Just delete the file from AP.

Michelle