every vote an IP

[omidMolaverdi]

Hi,

Thank you for your package. I encountered an issue that blocks my way forward.

It seems that you have locked an IP one vote however, there should be a freedom of vote interval for voting otherwise it won't work for the websites proxied by e.g. Cloudflare that the IP addresses are limited.

I normally cope with this issue by defining a freedom of voting variable and set it to a value like 1 hour. In your case it would be rewriting the following function with the added extra line:

public function hasVoted($poll_id,$freedomofvoteHours )
    {
        $poll = Poll::findOrFail($poll_id);

        if ($poll->canGuestVote()) {
            $result = DB::table('larapoll_polls')
                ->selectRaw('count(*) As total')
                ->join('larapoll_options', 'larapoll_polls.id', '=', 'larapoll_options.poll_id')
                ->join('larapoll_votes', 'larapoll_votes.option_id', '=', 'larapoll_options.id')
                *******->whereRaw('larapoll_votes.created_at >= DATE_ADD(NOW, INTERVAL -' . $freedomofvoteHours . ' HOUR)')
                ->where('larapoll_votes.user_id', request()->ip())
                ->where('larapoll_options.poll_id', $poll_id)->count();
            return $result !== 0;
        }

        return $this->options()->where('poll_id', $poll->id)->count() !== 0;
    }

The other difficult policy in this package is to edit the options by the admin. Obviously, there may be always typos and your current policy does not allow any adjustment to the option after the first vote.

[mrwhale]

I encountered something similar a little while ago as I had the need for any user to be able to vote, rather then basing it on IP As this poll feature I used was only published to target 1 company who resided in 1 office (so everyone had the same external IP)

I got around that by using the request session rather then the request IP

#80 (comment)

edit:
Also note that I had to change the Guest model so the Guest ID was the session token too
Guest.php

    public function __construct(Request $request)
    {
            $this->user_id = $request->session()->token();
    }

[akiyamaSM]

I'll be working on that for the next release.

[omidMolaverdi]

Hi

@akiyamaSM Just a touch base, do you plan to fix this?

[akiyamaSM]

@omidMolaverdi Not any soon. I'd be happy to see a PR from you.