Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
see full discussion here:
http://drupal.org/node/64861
this simple patch will render registering phony accounts absolutely pointless for spammers.
Comment | File | Size | Author |
---|---|---|---|
#6 | spam.diff_0.txt | 633 bytes | beginner |
#2 | spam.diff.txt | 645 bytes | beginner |
registration-spam.diff.txt | 634 bytes | beginner |
Comments
Comment #1
killes@www.drop.org CreditAttribution: killes@www.drop.org commentedThe patch does not comply with coding standards: We use || not OR.
Comment #2
beginner CreditAttribution: beginner commentednew patch.
I see the different precedence between &&, || and AND, OR.
http://www.php.net/manual/en/language.operators.logical.php
Is there any particular (historical) reason why Drupal must use && and || instead of AND and OR?
Comment #3
drummThis doesn't seem to have much to do with "meta roles"; what does this patch do, concisely?
Comment #4
beginner CreditAttribution: beginner commentedthe patch is relevant to the discussion about spam registration that I addressed on the 'meta-roles' page:
This page removes that incentive, because an account that is never activated (because created by a spammer with a fake email or an email they don't own) is not accessible by anonymous or normal users, and most importantly from the spammers' point of view, the page is not accessible by search engines...
Comment #5
killes@www.drop.org CreditAttribution: killes@www.drop.org commentedapplied to 4.7, downgrading as nothign is that broken.
Comment #6
beginner CreditAttribution: beginner commentedclean patch for head.
Is there any particular (historical) reason why Drupal must use && and || instead of AND and OR?
Comment #7
beginner CreditAttribution: beginner commentedComment #8
beginner CreditAttribution: beginner commentedComment #9
killes@www.drop.org CreditAttribution: killes@www.drop.org commentedWe have decided to use && and || and want to keep it consistent.
Comment #10
beginner CreditAttribution: beginner commentedthanks :)
Comment #11
drummCommitted to HEAD.
Comment #12
(not verified) CreditAttribution: commented