User gets registered with unchecked Terms

  • Resolved klm12

    (@klm12)


    10 years, 2 months ago

    Hello,

    First of all, thanks for providing this great plugin.. Prevents hacking in wp-login.php …
    I use WordPress 3.9.2. with other Login-related Plugins like “New User Approve” and “Limit Login Attempt” that I might refer here to.
    On the register page the link to the terms of use appears. Also the checkbox and the error message on top when it’s not checked and the form submitted…

    But besides the error message being displayed, in the background the username and e-mail is inserted into the database.

    The user immediately sees that, after checking the checkbox and submitting the form again: “Username and/or email is already registered” is displayed!!

    Well, do you know a fix for that? That the Input of the form is not entered into the database (and email to admin is sent out…) until the checkbox is checked?
    The simplest thing I was thinking about is a client based javascript check, but would not have sth to do with your plugin.. So if you could fix that there or give me hints what might be wrong?

    Thanks!

    https://www.ads-software.com/plugins/agreeable/

Viewing 7 replies - 1 through 7 (of 7 total)
  • Plugin Contributor kraftpress

    (@kraftpress)

    Responded to your email as well ??

    Are you running Multisite or BuddyPress?

    Thread Starter klm12

    (@klm12)

    Hey, yea maybe lets discuss it here, better for other users!

    So I neither run BuddyPress nor a Multisite.
    Just a standard single WordPress 3.9.2 installation!

    I think somehow when a user doesnt check the checkbox it should have the same function like if he doesn’t enter a email or name.
    The given errors are correct, but the transaction is not prevented from happening, which results in a new DB-entry.
    When u don’t enter a username or email, this doesnt happen with the given errors

    Plugin Contributor kraftpress

    (@kraftpress)

    Thank you for the update, we’re looking into it!

    Plugin Contributor kraftpress

    (@kraftpress)

    Pushed an update that should resolve this issue! Let me know if everything is working for you after the update ??

    Thread Starter klm12

    (@klm12)

    Hey,
    thank you. When I have time I will check whether it resolved the issue!

    So far I’ve done a small clientsided (popup) solution without using your plugin. Just in case anybody is interested (changes in the wp-login.php):

    Put this code within the <head> area:

    <!-- Client based jquery javascript which prevents form from being submitted when checkbox is not checked and displays error message
	see later down in form RegisterForm -->
	<script src="https://ajax.googleapis.com/ajax/libs/jquery/1.11.1/jquery.min.js"></script>
	<script type='text/javascript'>

	$(document).ready( function() {

		function checkCheckbox() {

			if ( $('#termsOfService').is(':checked') )
			{
				return true;
			}

			else
			{
				alert('Please accept our Terms of Service!');
				return false;
			}

		}

		$('#registerform').submit( checkCheckbox );

	} );
	</script>

    and this within the <body> area and in the <form name=”registerform […] > just where you want to have a checkbox

    <!-- Added Checkbox with termsOfService -->
	<p><input id="termsOfService" type="checkbox" /><?php _e('I have read the <a href="'); echo get_bloginfo('url'); _e('/termsOfServicePageUrl/" target="_blank">Terms of Service</a> and accept them.') ?></p>
    Plugin Contributor kraftpress

    (@kraftpress)

    Yup that’ll work! Though of course editing WordPress core files is a recipe for disaster, and javascript verification is easily bypassed, so make sure no one tries that for a long term solution! ??

    Hey, I use rtmedia plugin and agreeable both in my buddypress website.Both are working well in on there place.but when I active both together rtmedia light box is not working.Because i am using rtmedia pro version they give me premium support.and told me that “It seems like that ‘agreeable’ plugin has some JS error that is not allowing lightbox to open. See attached image.
    You may need to fix that js error or get in touch with plugin author”.

Viewing 7 replies - 1 through 7 (of 7 total)
  • The topic ‘User gets registered with unchecked Terms’ is closed to new replies.