This PR is related to issue #39 and is a suggestion to allow easy Validator mocking:

HoneypotValidator::shouldReceive('validateHoneytime')
    ->once()
    ->andReturn(true);

I have updated the documentation.

This fixes cases where the honeypot marco is not added where the form builder hasn't been resolved through the IoC, i.e. if your extending Laravel's form builder

Honeypot is working fine in my localhost using windows

but after i upload my update in my server "Linux"

1. composer.json
2. vendor/msurguy/ all files

Then when i access my website i have the error below: FatalErrorException in ProviderRepository.php line 150: Class 'Msurguy\Honeypot\HoneypotServiceProvider' not found

Thanks In advanced

Is there a release for Laravel 5 yet? I'm trying to upgrade and just using the normal install on the ReadMe causes an error due to HTLM being deprecated.

PHP Fatal error: Class 'Illuminate\Html\FormBuilder' not found in /vagrant/biospex/vendor/msurguy/honeypot/src/Msurguy/Honeypot/HoneypotServiceProvider.php on line 91

I'm getting the following error using Laravel 5.0.31:

FatalErrorException in HoneypotServiceProvider.php line 91:
Class 'Illuminate\Html\FormBuilder' not found
in HoneypotServiceProvider.php line 91
at FatalErrorException->__construct() in HandleExceptions.php line 131
at HandleExceptions->fatalExceptionFromError() in HandleExceptions.php line 116
at HandleExceptions->handleShutdown() in HandleExceptions.php line 0
at HoneypotServiceProvider->registerFormMacro() in HoneypotServiceProvider.php line 57
at HoneypotServiceProvider->Msurguy\Honeypot\{closure}() in Application.php line 862
at call_user_func:{/Users/mario/workspace/laravel/vendor/laravel/framework/src/Illuminate/Foundation/Application.php:862}() in Application.php line 862
at Application->fireAppCallbacks() in Application.php line 690
at Application->boot() in BootProviders.php line 15
at BootProviders->bootstrap() in Application.php line 183
at Application->bootstrapWith() in Kernel.php line 199
at Kernel->bootstrap() in Kernel.php line 110
at Kernel->sendRequestThroughRouter() in Kernel.php line 84
at Kernel->handle() in index.php line 53
in server.php line 21
at {main}() in server.php line 0

@msurguy how do you feel about this one? Of course a breaking change so would need a major version bump but I think currently a lot of issues spawn from usage of the macro on FormBuilder. These issues range from supporting legacy Laravel < 4.2, newer Laravel >= 4.2.. and now that illuminate/html has been deprecated it's been replaced with another package called Laravel Collective which runs under a different namespace but is essentially the new illluminate/html.

That's a lot to support and keep up with when all it's doing is just adding a macro which isn't really worth it imo. Doing Honeypot::getFormHtml(...) vs Form::honeypot(...) isn't really much of a difference.

As this is a breaking change, I've also added in this commit a rename of the method getFormHtml to simply generate -- it's possibly a nice time to make the change as read's nicer in it's usage:

{{ Honeypot::getFormHtml() }} {{ Honeypot::generate() }}

Let me know what you think :smile:

Service Provider boot happens before anything else. You're loading translated messages here: https://github.com/msurguy/Honeypot/blob/master/src/Msurguy/Honeypot/HoneypotServiceProvider.php#L19

So if I add

App::setLocale('another lang');

to my app/start/global.php, it's too late, because the english (default) was already loaded.

I have two of exactly the same forms on a page, which both include the same Honeypot::generate() method. When doing an HTML validity check (https://validator.w3.org) It says there are duplicate ids on the containing div ..._wrap.

1. Is this wrapper necessary?
2. Is there a way around using the same id on this element

The class name for the DecryptException has changed in L5. Instead of: \Illuminate\Encryption\DecryptException

it is now: \Illuminate\Contracts\Encryption\DecryptException

So in order to support both L4 and L5 integrations I suggest to change the catch to a generic Exception class.

When a bot throws bad data at your form then the decrypt for the honeytime will throw a Illuminate\Encryption\DecryptException exception which does not get caught. I wrapped my validation in a try catch to handle it on my end for now, but it would be nice if this was handled internally.

I followed the steps up until adding the honeyPot to the markup pages or in other words ('view') but i don't understand where you add the validation to the HoneyPot ? Do you add it to the controller ? or you add it to the route ? can you please explain. I am doing a demo so after i ran artisan make:auth on a clean installation of Laravel 5.2.41 I added honey pot the main registration form. So i am thinking to add it to App\Http\Controllers\Auth\AuthController.php since the validator is already being used in that class. but i am not sure if thats the right way ? because i wanna also check the time of form being filled too. can someone please advise.

thanks

Ran the installer and updated package.json to dev-master but getting this error: [ErrorException]
Array to string conversion

Versions: PHP 7.1.7 Composer version 1.5.6 laravel/framework: 5.2.*

I'm trying to extend Honeypot, and I'm running into issues.

I created my service provider, and that works fine, but I can't seem to call the parent class methods.

<?php

namespace App\Extensions;

use Honeypot;

class MyHoneypot extends Honeypot
{
/**
 * Generate a new honeypot and return the form HTML
 * @param  string $honey_name
 * @param  string $honey_time
 * @return string
 */
public function generate(array $honey_name, array $honey_time)
{
    $honey_time_encrypted = parent::getEncryptedTime();
    // Encrypt the current time

    $html = '<div id="' . $honey_name . '_wrap" style="display:none;">' . "\r\n" .
                '<input name="' . $honey_name . '" type="text" value="" id="' . $honey_name . '"/>' . "\r\n" .
                '<input name="' . $honey_time . '" type="text" value="' . $honey_time_encrypted . '"/>' . "\r\n" .
            '</div>';

    return $html;
}
}

I get the error Call to undefined method App\Extensions\MyHoneypot::getEncryptedTime().

I also attempted to use Msurguy\Honeypot, but that, too, failed. It could not find the class.

Any thoughts why?

Hi, thank you for this nice package which works nicely.

I am using it in a registration form and want to add an integration test to make sure the form can be filled and submitted properly. However, the form is filled too fast by my tests and the validation fails. I am looking for a way to mock the HoneypotValidator->validateHoneytime() method and make it return true during my test.

Is there a way to do this ?

Thank you

Might just be me but I'm getting: An exception has been thrown during the rendering of a template ("Method honeypot does not exist.") in my view. I'm using Twig and access your method as such, form_honeypot('hname', 'htime').

I also have my own custom macros which I'm able to access as form_mymacroname('xxx') so I don't think it's the macro but I might be wrong on this.