Update for multiple app('captcha')->display() on the same page #57

seltix5 · 2016-12-31T21:36:25Z

with this update will be possible to just call the noRecaptcha on the same page without requesting the api multiple times, just like this :

app('captcha')->display()

registerin the service provider with singleton instead of bind will return always the same object per page load. with this methos is possible to insert a control variable in the class and use it to call the api only once.

ading this var control will permit to request the api only once. this will work with the singleton register on te serviceprovider file.

anhskohbo · 2017-01-01T00:22:21Z

Thanks for pull request, pls check my code review.

anhskohbo · 2017-01-02T09:57:07Z

I need test this pr before make a new release

seltix5 · 2017-01-02T10:08:05Z

sure ^^

anhskohbo · 2017-01-04T02:34:45Z

Hi seltix5, could you give me a example about multiple widgets in a page?

See: https://developers.google.com/recaptcha/docs/display

seltix5 · 2017-01-04T09:21:54Z

hi!
so, after furder checking this situation, i forget that the default google reCaptcha code will automatically render only the first div/instance, so i added a function to use the "explicit" mode, with this changes to request only one noCaptcha just call display :

{!! app('captcha')->display() !!}

when usign multipla instance we should call only the functio multiple_display like this :

{!! app('captcha')->multiple_display() !!}
.....
{!! app('captcha')->multiple_display() !!}

what do you think?

#58

anhskohbo

.

seltix5 added 2 commits December 31, 2016 21:33

Update for multiple app('captcha')->display() on the same page

87e8fcb

registerin the service provider with singleton instead of bind will return always the same object per page load. with this methos is possible to insert a control variable in the class and use it to call the api only once.

Upda for multiple noCaptcha on the same page

8269705

ading this var control will permit to request the api only once. this will work with the singleton register on te serviceprovider file.

seltix5 mentioned this pull request Dec 31, 2016

Multiple rendering of no-captcha in the same page #10

Closed

seltix5 added 2 commits January 1, 2017 00:25

Update NoCaptcha.php

df27e2d

Update NoCaptcha.php

392ce29

anhskohbo merged commit f5befe2 into anhskohbo:master Jan 1, 2017

anhskohbo reviewed Jan 5, 2017

View reviewed changes

anhskohbo added a commit that referenced this pull request Jan 30, 2017

Rollback #57

81675cc

FrederickFarfan mentioned this pull request May 5, 2017

file_get_contents(https://www.google.com/recaptcha/api/siteverify): failed to open stream: HTTP request failed! HTTP/1.1 403 Forbidden #75

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Update for multiple app('captcha')->display() on the same page #57

Update for multiple app('captcha')->display() on the same page #57

seltix5 commented Dec 31, 2016

anhskohbo commented Jan 1, 2017

anhskohbo commented Jan 2, 2017

seltix5 commented Jan 2, 2017

anhskohbo commented Jan 4, 2017

seltix5 commented Jan 4, 2017

anhskohbo left a comment •

edited

Loading

Update for multiple app('captcha')->display() on the same page #57

Update for multiple app('captcha')->display() on the same page #57

Conversation

seltix5 commented Dec 31, 2016

anhskohbo commented Jan 1, 2017

anhskohbo commented Jan 2, 2017

seltix5 commented Jan 2, 2017

anhskohbo commented Jan 4, 2017

seltix5 commented Jan 4, 2017

anhskohbo left a comment • edited Loading

Choose a reason for hiding this comment

anhskohbo left a comment •

edited

Loading