Posted by jflesher on May 23, 2011 at 8:13pm
4 followers
| Project: | CAPTCHA Pack |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | needs work |
Issue Summary
Do you have plans to migrate this to Drupal 7?
If not, what is the Drupal 7 way to handle CAPTCHA?
Comments
#1
I am the current maintainer of CAPTCHA Pack and do this in my free time.
Currently, I have unfortunately no spare time left to invest in maintenance or development.
So it is very unlikely that a Drupal7 port will happen in te near future, unless a new (co)maintainer jumps in.
Apart from that, there is an initial port of the core CAPTCHA module for Drupal 7 and there is also the mollom module
(I have updated the maintenance status on the project page for clarity)
#2
Hi everybody...
i ported the modules to drupal 7 - at least those modules that worked in the drupal 6-version
Currently not working
Those were not working in the 6.x-dev version and i am new to the drupal7-form-api so there will be more patches as soon as i get them to work.
Regards,
Jens
#3
Just some comments.
+++ b/ascii_art_captcha/ascii_art_captcha.infoundefined@@ -2,4 +2,9 @@ name = "ASCII art CAPTCHA"
+; Files
+files[] = ascii_art_captcha.module
+files[] = ascii_art_captcha.admin.inc
+files[] = ascii_art_captcha.install
You only have to add files to the .info file if the files has class/interface in it. This might be a bug in the coder module.
+++ b/ascii_art_captcha/ascii_art_captcha.installundefined@@ -1,6 +1,7 @@
+// $Id$
That's not needed anymore after the git migration.
+++ b/text_captcha/phrase_captcha/phrase_captcha.incundefined
diff --git a/text_captcha/phrase_captcha/phrase_captcha.info b/text_captcha/phrase_captcha/phrase_captcha.info
index 3e7dcd3..1fd7f4e
index 3e7dcd3..1fd7f4e
--- a/text_captcha/phrase_captcha/phrase_captcha.info
--- a/text_captcha/phrase_captcha/phrase_captcha.info
+++ b/text_captcha/phrase_captcha/phrase_captcha.infoundefined
In general it's much easier to apply a patch if it's a diff against the same file.
Powered by Dreditor.
+++ b/text_captcha/word_list_captcha/word_list_captcha.moduleundefined@@ -90,7 +90,7 @@ function word_list_captcha_captcha($op, $captcha_type='', $post_data=array()) {
- drupal_add_css(drupal_get_path('module', 'word_list_captcha') .'/../text_captcha.css');
+ drupal_add_css(drupal_get_path('module', 'word_list_captcha') . '/../text_captcha.css');
There seems to be a form. You can attach the css with $form_element['#attached']['css'] = array('jsfile' => array());
When you do it like this, it works on cached forms/ajax as well.
+++ b/ascii_art_captcha/ascii_art_captcha.infoundefined@@ -2,4 +2,9 @@ name = "ASCII art CAPTCHA"
+; Files
+files[] = ascii_art_captcha.module
+files[] = ascii_art_captcha.admin.inc
+files[] = ascii_art_captcha.install
You only have to add files to the .info file if the files has class/interface in it. This might be a bug in the coder module.
+++ b/ascii_art_captcha/ascii_art_captcha.installundefined@@ -1,6 +1,7 @@
+// $Id$
That's not needed anymore after the git migration.
+++ b/text_captcha/phrase_captcha/phrase_captcha.incundefined
diff --git a/text_captcha/phrase_captcha/phrase_captcha.info b/text_captcha/phrase_captcha/phrase_captcha.info
index 3e7dcd3..1fd7f4e
index 3e7dcd3..1fd7f4e
--- a/text_captcha/phrase_captcha/phrase_captcha.info
--- a/text_captcha/phrase_captcha/phrase_captcha.info
+++ b/text_captcha/phrase_captcha/phrase_captcha.infoundefined
In general it's much easier to apply a patch if it's a diff against the same file.
Powered by Dreditor.
#4
Hi,
thanks a lot for the comments... I'm totally new to drupal-module-patching ;-)
Will try to make it better in the next version. (Have to work out the issues with the forms in the not working captcha-types...)
Jens
#5
You just have to
a) git clone the module
b) make a new branch via git checkout -b newbranch
c) do the work against this branch git diff OLDBRANCH > foo.patch
That's all