msurguy / Honeypot

Simple spam prevention package for Laravel applications
maxoffsky.com/code-blog/implementing-honeypot-spam-prevention-in-laravel-applications/
MIT License
429 stars 44 forks source link

Extending Honeypot #41

Open hoppities opened 8 years ago

hoppities commented 8 years ago

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?

garygreen commented 8 years ago

Try the following:

namespace App\Extensions;

use Msurguy\Honeypot\Honeypot as BaseHoneypot;

class Honeypot extends BaseHoneypot
{

    public function generate(array $honey_name, array $honey_time)
    {
        $honey_time_encrypted = parent::getEncryptedTime();
        //
        //....
        //....
    }

}

You will also need to replace the resolved instance in the IOC. A good place to put this is in your own service provider which is included after HoneypotServiceProvider in your app.php:

app()->singleton('honeypot', 'App\Extensions\Honeypot');

If you have any further questions it's probably best asking on Stackoverflow as this stuff isn't really specific to honeypot but more of an understanding of namespaces and Laravel's IOC.

hoppities commented 8 years ago

Ah! Of course...thanks!