Hi there,

The module is not compatible with filefields - if they are filled then the report still says they are not when checking. Is there anyone who can urgently help with this for a bounty of $100 or maybe a bit more? Need something within 36 hours!!! (Long shot I know)

Thanks,

Adam

CommentFileSizeAuthor
#2 wf_required_fields-filefield_compatibility_898366_1.patch1.25 KBWorldFallz

Comments

WorldFallz’s picture

WorldFallz commented

i think i can do this for you in about an hour... use my contact form and let me know if you're interested.

WorldFallz’s picture

WorldFallz commented
Status: Active » Needs review
StatusFileSize
new wf_required_fields-filefield_compatibility_898366_1.patch1.25 KB

attach is a patch that adjusts the validation for filefields.

adshill’s picture

adshill commented

Works like a dream. WorldFallz, thanks a lot. I'm sending you an e-mail now (also with another potential bounty!) and the $100 bounty is on its way.

I will leave this as needs review for now as its probably a good idea for someone else to give it a try but it will be used extensively over the next weeks so I'll try and post again after that. Thanks again.

kbk’s picture

kbk commented

I can confirm this patch works as advertised.

Prior to patch: A CCK FileField designated as required by WFRF is flagged as empty regardless of whether or not a file has been uploaded.

Following patch application: A CCK FileField designated as required by WFRF is flagged as empty until a file is uploaded; after file upload the field is no longer flagged.

I can't comment on the code itself so will let someone else move to RTBC.

kbk’s picture

kbk commented
Version: 6.x-1.0 » 6.x-1.x-dev
Status: Needs review » Reviewed & tested by the community

Let's get this into dev please!

deekayen’s picture

deekayen commented
Status: Reviewed & tested by the community » Fixed

committed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.