Kdyby / Google

[DEPRECATED] Use https://github.com/thephpleague/oauth2-google instead
Other
24 stars 35 forks source link

isUrl & nette 2.3 #12

Closed Keeehi closed 9 years ago

Keeehi commented 9 years ago

Nová verze nette nejspíše změnila způsob jak pracuje s konfiguračními soubory. Nevím jestli je to BC break nové verze, nebo si to má hlídat Kdyby/Google nicméně vznikl problém

google:
    returnUri: "Server:googleLogin"(do=googleLogin-response)

A výsledkem je

Warning
preg_match() expects parameter 2 to be string, object given
// in Kdyby/Google/DI/GoogleExtension.php:164

Jako druhý argument se totiž do metody předá

Nette\DI\Statement #01
    entity private => "Server:googleLogin"
    arguments => array (1)
        do => "googleLogin-response"

Když se v konfiguraci odmaže ten signál, tak už to projde. Problém bude zase ale při návratu od goolu.

fprochazka commented 9 years ago

@Keeehi thank you for reporting, can you please verify now that it's fixed?

Keeehi commented 9 years ago

@fprochazka Sure. It works. dev-master version installed trough composer. Thanks for quick fix.