Closed (fixed)
Project:
Content Profile
Version:
6.x-1.x-dev
Component:
Base module
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
17 Oct 2008 at 19:35 UTC
Updated:
19 Jan 2009 at 20:00 UTC
Jump to comment: Most recent file
Comments
Comment #1
jon pughCoverted 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:
not sure if the above it appropriate for the sleek drupal6 include protocols, but its a start
Comment #2
fagoCould you post this as a proper patch please? Are there any panels users available for testing? :)
Comment #3
jon pughComment #4
fagothanks.
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.. :)
Comment #5
fagoThe code of Michelle (http://drupal.org/node/350161) looks like being the same, so I've just committed this. Please test.
Comment #6
michelleDoh! 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
Comment #7
niklp commentedw000t! now if I can just get all of panels 2 to work... :p