Any plans to upgrade this module. Drupal 6 probably only has about another 10 months of life left in it.

We can probably help with testing.

CommentFileSizeAuthor
#7 webform_represent-d7-1922806-7.patch14.79 KBdtarc
#6 webform_represent-d7-1922806-6.patch20 KBdtarc
#4 webform-represent-d7-1922806-4.patch9.25 KBbenstjohn
#3 webform-represent-upgrade-1922806-3.patch1.63 KBmgifford

Comments

CatherineOmega’s picture

CatherineOmega commented

I would definitely be interested in testing this. And if anyone's already working on it, I may be able to arrange some funds to chip in for development.

jpmckinney’s picture

jpmckinney commented

I'm not familiar with how Webform 7.x differs from 6.x, and we don't have any clients asking for a D7 version, so it's unlikely we would build a D7 version. On the other hand, we would merge anyone's patch for a D7 version.

mgifford’s picture

mgifford
he/him
Primary language English
commented
Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new webform-represent-upgrade-1922806-3.patch1.63 KB

It's by no means ready, but it's a first kick of the can. I wanted to look at how this was done here to look at bringing over to https://drupal.org/project/citcon

benstjohn’s picture

benstjohn commented
StatusFileSize
new webform-represent-d7-1922806-4.patch9.25 KB

Recently wrote this little patch that fixes the ajax part in drupal 7.

mgifford’s picture

mgifford
he/him
Primary language English
commented

Thanks @benstjohn - we're in the midst of building https://www.drupal.org/project/engage and including the Represent API in that.

Thanks for contributing the patch though. Hope it gets in an update.

dtarc’s picture

dtarc commented
StatusFileSize
new webform_represent-d7-1922806-6.patch20 KB

Here's another D7 patch. Functionality is a bit different though, much less JS & representataives no longer divided by set.

dtarc’s picture

dtarc commented
StatusFileSize
new webform_represent-d7-1922806-7.patch14.79 KB

Here's another patch, which does not remove the binary gif file

mgifford’s picture

mgifford
he/him
Primary language English
commented

Patch applies nicely. I haven't had a chance to test it with Webform & D7.

jpmckinney’s picture

jpmckinney commented

Hi @dtarc, I've added you as maintainer, so feel free to apply the patch and create a release. Thanks for pinging me, @mgifford!

dtarc’s picture

dtarc commented

Thanks for the commit access. I have started a 7.x-2.x branch and I'll tag an alpha soon.

Also I'll roll the patch from #4 into a 7.x-1.x branch.

dtarc’s picture

dtarc commented
Assigned: Unassigned » dtarc
Status: Needs review » Needs work
lambic’s picture

lambic commented
Status: Needs work » Fixed
lambic’s picture

lambic commented
Status: Fixed » Closed (fixed)