From talking with Michelle briefly it sounds like this is already in the works... but filing a FR here so others can track progress and pitch in where & when possible.
Cheers :)
Scott

Comments

jon pugh’s picture

Status: Active » Needs review
StatusFileSize
new1.65 KB

Coverted old nodeprofile_panels.inc to content_profile.panels.inc...

Did a quick test and it seems to work fine, just like the old nodeprofile panels2 functionality!

Had to add an include at the top of content_profile.module:


if (module_exists('panels')) {
   module_load_include('inc', 'content_profile', 'content_profile.panels');

}

not sure if the above it appropriate for the sleek drupal6 include protocols, but its a start

fago’s picture

Status: Needs review » Needs work

Could you post this as a proper patch please? Are there any panels users available for testing? :)

jon pugh’s picture

Status: Needs work » Needs review
StatusFileSize
new2.56 KB
fago’s picture

thanks.

You have some tabs in your patch in content_profile_init(). Also please move hook_init() somewhere to the top of the module. Anyway we need a review by a panels user now.. :)

fago’s picture

Status: Needs review » Fixed

The code of Michelle (http://drupal.org/node/350161) looks like being the same, so I've just committed this. Please test.

michelle’s picture

Doh! I had no idea this was here until you pointed it out on IRC. There was no support in the module itself so I just assumed you hadn't gotten around to it yet. Never occured to me that someone else would have done the same thing I did. LOL

Great, glad it's added. One less patched module I have to give my client today. :) Now if I can just get CCK to commit the fieldgroup code....

Michelle

niklp’s picture

w000t! now if I can just get all of panels 2 to work... :p

Status: Fixed » Closed (fixed)

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