I rewrote the profile.module implementing a profile_hook that allows any modules to extend the field types from the profile.
It doesn't fully support the browser feature yet - it just supports the basic fields (i.e the ones implemented by the profile module).
It is fully compatible with the old module.

In all my tests it worked well, please review and test it. As soon I receive an Ok, I will start to upgrade it to the Drupal head profile.module.

CommentFileSizeAuthor
profile_10.module33.7 KBwundo

Comments

tanepiper’s picture

Whats the chances of getting this to work in the HEAD version (5.x)? I had a look using winmerge but unfortunatly, the code isn't commented to say what you have added and removed and I don't just want to start merging code if you have used the old API.

This patch will be very handy for me, as I am looking at building a hCard generator from the profile fields, and this feature is exactly what I need before I start going off and building it myself.

wundo’s picture

Whats the chances of getting this to work in the HEAD version (5.x)?

As I said, as far someone reviews it, I will start to port it to cvs HEAD...

ardas’s picture

I've looked through the code and compare with the core profile.module.
It is very clever idea to use hooks in profile. This will give everybody an ability to create specific fields. I think now that if even CCK fields won't be supported by profile module we can base profile field modules on CCK modules without any problems, or create patches for CCK module which will implement new profile API.

I have updated my old profile module in the current project and am going to test your one and study it. I'll report you about my results shortly.

Thanks a lot.
I'm eager to take part and help with this module.

killes@www.drop.org’s picture

Version: 4.7.4 » 6.x-dev

no new features in 4.7

ardas’s picture

Hello,

I'm testing new profile module. It looks good. Create/Edit/Save operations look working correctly.

I have one question for now: why did you decide to reorder form fields? I noticed Page title field and Selection options (for List field) were moved on the top of the form. Were there any special reason or you just missed the Order attribute?

Thanks.

ardas’s picture

By the way, I did some propositions concerning profile.module but looks like its original author don't care too much or very busy with other work.
You may find them useful for the new version you create ... and they are extremely easy to implement :).

1. The first one is an ability to customize keys for lists described here http://drupal.org/node/89468.
2. The second very small and a simple issue is a small range for profile fields Weight attribute http://drupal.org/node/92343. Just change delta to 50 instead of 5.

If you are developing a new profile module you may include all useful features to improve it.
Thanks.

ardas’s picture

Looks like a bug with list selection field.
1. Create a list with two items 0 and 1.
2. Save this field.
3. Edit this field.
ISSUE: Selection options text area is empty dispite the fact that options were saved into the database.

wundo’s picture

Folks,
Sorry for the delay in replying you, as I said for the folks in the Captcha thread I'm really busy this week, but I'm planning to work more on my Drupal projects next week.

Thanks for the reviews, and please give me feedbacks, I'm reading all of them...
cheers,
fabiano.

wundo’s picture

Version: 6.x-dev » 4.7.x-dev

Hey Folks
Following the discussion I had in PVT with Duke, I'm starting this issue as an module to Drupal 4.7 (such kind of a fork of core's profile module).
Here is the new project home page: http://drupal.org/project/profile-ng

Cheers,
Fabiano.

wundo’s picture

Status: Needs review » Closed (fixed)
moshe weitzman’s picture

how are these hooks better/different form the ones already provided within hook_user(). just wondering.