It seems that a lot of features I want are missing, so I started patching your module (started on a 5.x-1.0 version) to add this ones:
* possibility to limit registration form configuration to make it simplier, this implies to set default values in administration page, and restrict the node creation/edit form
* possibility to allow drupal users/others/both to register for a event, see http://drupal.org/node/232158

If you're interrested in my code, I'll submit here my patch when it will be finished

Comments

pounard’s picture

StatusFileSize
new47.44 KB
new35.33 KB

Here some screenshots of what I am doing, please consider I will send my patch only when it's finished (few days I think).

pounard’s picture

Status: Needs review » Active
StatusFileSize
new136.32 KB

I finished my patches against 5.x-1.0 version (and ONLY this version), you should check.
Despite the fact it works without problems on my configuration, this patch need a lot of testing I think.

To install and use it:

cd /your/drupal/sites/all/modules
wget http://ftp.drupal.org/files/projects/event_manager-5.x-1.0.tar.gz
tar xzfm http://ftp.drupal.org/files/projects/event_manager-5.x-1.0.tar.gz
wget <thispatchurl>/event_manager-5.x-1.0-drupal_users-restrict_admin.patch
patch -p0 < event_manager-5.x-1.0-drupal_users-restrict_admin.patch

Further, your code is OK, but now, with this support I had to divide big functions to a set of small ones to factorize the code as much as possible, in the participants loading for example, cause the data can now come from many places (drupal users, or custom personal info form).

Some parts of the code are still dirty I think, but it works well, please, maintainers, check this and give a feed back. If this set of patches won't be integrated to the current version, I'll must maintain my version on my own because this patches target is run on a production environment.

Current features for users are:
- possibility to reduce editing form, and to restrict it to a very simple one (via the admin settings)
- possibility to register drupal users in 'one click way'
- some bugfixes are done (not much)

Some features for dev are:
- some code factorisation helps the maintainability (not much too, but mandatory things are done to allow drupal users to register)

pounard’s picture

Status: Active » Needs review
pounard’s picture

Status: Active » Needs review
StatusFileSize
new139.45 KB

I made bugfixes, see new patch.
EDIT: changed patch file with one with more bugfixes

SpriteGF’s picture

Pounard, your work is incredible. Have you considered taking up maintenace of this module or taking this code and rolling it into your own module? It seems like the module was a one-time creation and the original authors don't have the time to maintain this.

sun’s picture

Status: Needs review » Closed (won't fix)

Sorry to do this to you, but please separate all changes into atomic patches and create a separate issue for each topic. No one is able to review a 139 KB patch that contains coding-style changes and implements "miscellaneous" features.

pounard’s picture

@SpriteGF
I did it cause I did need it, but I have to admit that finally, I did a short module that respond in a far better way to my need. The fact is I did abandon bugfixing this patch. This module's mainteners (if their are some left) could use as a sample, if some features they want are included in my changes.

@sun
Ok, I understand that, I'll maybe have time later to do it, but not right now. As I said just over there, with the lack of response and my short timeline I had to choose a more rational way to get what I need than contribute to this module, sorry.

chrispooh’s picture

Hi,

it is a great modul. I find the way to add extra fields to registration form. Have you idea to do this?

Best Regards,
Chris