Should this module merge with the UC_Webform_Pane module? It seems like UC_Event_Registration solves a subset of functionality that UC_Webform_Pane has and could possibly help it.

Comments

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented

FYI I opened a corresponding ticket for UC_Webform_Pane: #1445488: Merge with UC_Event_Registration?

arski’s picture

arski CreditAttribution: arski commented

I don't see any added value from doing that - UC Event Registration seems to be very specific, plus it creates a new content type, a new webform, and requires two webform fields, none of which is a requirement in UCWP and really probably never will.

At the end of the day, trying to integrate a module like UCER, which as the maintainers say will probably not even have a 7.x version, into UCWP will just take loads of time and have little benefit.. so from this end I don't really see any use, sorry.

Keeping the discussion here btw, as this is more of a subset of UCWP (kind of). Closed the issue in UCWP.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented

I've suggested several tickets that would help clean up the existing module:

Also:

Once these are done (I may be providing some patches for them) the module will still retain the existing functionality but would be much more reusable and generic, and definitely overlap more with UC_Webform_Pane.

arski’s picture

arski CreditAttribution: arski commented

Great. well, maybe you'll be able to use ucwp as a basis for some of that and hook into that to do your stuff - I think that might be reasonable and it will allow you to ignore the webform side of things, especially on checkout and order editing.. anyway good luck.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented

Thanks. One of the key requirements I have, though, is to be able to easily change the fields per event, which UC_Webform_Pane makes more complicated. Anyway, I'll fiddle with them both and see what I can come up with.

arski’s picture

arski CreditAttribution: arski commented

BTW, if you're using content types already, by UC_CCK_Pane will be of more use to you? just a thought