Hi,

I have created a graphical version of the math CAPTCHA based on the Image CAPTCHA submodule, which I propose to be made a part of the CAPTCHA module. It is easy to use, its configuration almost the same as the Image CAPTCHA's, and it is harder to crack than both Image CAPTCHA and the orig math CAPTCHA. An image of an example CAPTCHA is uploaded. If you are interested, please contact me.

Regards
Belaselm

CommentFileSizeAuthor
#11 captcha-7.x-1.0-beta2-1670896.patch44.08 KBBelaselm
#9 graphmath_captcha.diff2.4 KBBelaselm
#7 graphmath_captcha_corrected.tgz63.82 KBBelaselm
#5 graphmath_captcha.tar_.gz64.82 KBBelaselm
GraphMath_Captcha.png15.29 KBBelaselm

Comments

rogical’s picture

rogical commented

Good! Why don't you create a sandbox and apply to be a module?

Belaselm’s picture

Belaselm commented

That's probably because I can't access the git repository. (I could, but it seems I did something wrong, and the repository refuses the connection.) Anyway, this captcha is almost identical in code to the image captcha, and the applying rules say that in this case I'm to contact the authors of the original module, and offer it as a contribution/patch.

rogical’s picture

rogical commented

Then you can try to make a patch or upload a tarball first. However, git is a must in drupal development.

wundo’s picture

wundo commented
Status: Active » Postponed (maintainer needs more info)

rogical, could you upload a patch?

Belaselm’s picture

Belaselm commented
Status: Postponed (maintainer needs more info) » Active
StatusFileSize
new graphmath_captcha.tar_.gz64.82 KB

Yes, I know, but I have just started using git, and I'm not perfect from it yet. For now, I'm attaching a tarball archive. I'll also have two more captchas soon, I'll try to make them as independent modules.

rogical’s picture

rogical commented

Tested, seems all well.

Belaselm’s picture

Belaselm commented
StatusFileSize
new graphmath_captcha_corrected.tgz63.82 KB

Sorry, I have left some obsolete parts in the code, like UTF-8 split, which is unnecessary for numbers. Here is a modified version without these.

rogical’s picture

rogical commented

New files seems can't to be made in patch.

Belaselm’s picture

Belaselm commented
StatusFileSize
new graphmath_captcha.diff2.4 KB

Sorry, as I wrote I'm still new in this business. Will this diff to the originally uploaded graphmath_captcha.tar_.gz file be enough?

Belaselm’s picture

Belaselm commented

I have created a sandbox project, and uploaded the files of this GraphMath CAPTCHA there. You can find it at http://drupalcode.org/sandbox/Belaselm/1676284.git/

Belaselm’s picture

Belaselm commented
StatusFileSize
new captcha-7.x-1.0-beta2-1670896.patch44.08 KB

I have created a git patch from the proposed graphmath_captcha submodule. I hope it will be OK.

soxofaan’s picture

soxofaan commented
Status: Active » Needs work

Hi,

sorry for the delayed answer. Looking at the patch of #11, I think there is way too much code duplication with the image CAPTCHA module. It would be pretty bad for long term maintainability to do it like this. Why didn't you reuse the functions from image CAPTCHA directly?

Belaselm’s picture

Belaselm commented

Thanks for the suggestion, I'll work on it when I'll have some time. I thought about making this captcha a part of the image captcha module, along with a configuration option to select it, but if we keep it a submodule, it can be used alongside the image captcha.

adams.garfield’s picture

adams.garfield commented

Gr8 work Belaselm. Captchas are pretty hard to code..

chris matthews’s picture

chris matthews commented
Version: 7.x-1.0-beta2 » 7.x-1.x-dev
Issue summary: View changes

Version change only

anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented

@Belaselm could you please add your implementation as merge-request for review?

As I'm not maintaining the 7.x branch, someone else will have to review this one then.