Note, at time of writing this patch *must* be applied to the D6 version of the module - #1088074: call to undefined function _eloqua_cron on line 54 eloqua.module.

Title does what it says on the tin. Patch to follow shortly. =)

CommentFileSizeAuthor
#1 1361630-1-user_profile_support.patch19.93 KBgreg.harvey

Comments

greg.harvey’s picture

greg.harvey
Primary language English
Location Occitanie, France
commented
Status: Active » Needs review
StatusFileSize
new 1361630-1-user_profile_support.patch19.93 KB

Patch attached, actually created by bellesmanieres for proper Git attribution purposes.

drupalninja99’s picture

drupalninja99 commented

This looks pretty good, I worry about the amount of effort it's going to take to port this to D7. I am thinking it would be better to have like an entity forms module. Eric Peterson from the entity api project said as much. For D7 wouldn't you want to be able to map any entity form to an eloqua form?

At this stage I am not sure this is worth committing to the eloqua project. It might need to live as a sandbox module. I don't want to have to support new D6 functionality that isn't also ported to D7 and this one would not be ported easily.

drupalninja99’s picture

drupalninja99 commented
Status: Needs review » Postponed

I am thinking we should postpone this one. It would be useful to some D6 users as a sandbox that I could reference from the project description.

drupalninja99’s picture

drupalninja99 commented
Issue summary: View changes

@greg.harvey do you have a D7 port for this?

jasonawant’s picture

jasonawant
he/him
Primary language English
Location New Orleans, USA
commented
Status: Postponed » Closed (outdated)

closed as outdated.