Forum Replies Created

Viewing 15 replies - 1 through 15 (of 26 total)
  • Thread Starter Xtremefaith

    (@xtremefaith)

    Judging from this line you have no intentions of including emails in your plugin:

    $oauth_identity['email'] = $result_obj['emails'][0]['value'];
    // PROVIDER SPECIFIC: Google returns an array of email addresses.
    To respect privacy we currently don't collect the user's email address.

    Unfortunately without at least using it for the moment I don’t know how else you can compare against current users:

    function wpoa_match_wordpress_user_by_email($oauth_identity) {
        $userid = email_exists( $oauth_identity['email'] );
        $user = get_user_by('id', $userid);
        return $user;
    }
    $this->wpoa_match_wordpress_user_by_email($oauth_identity);

    If this is the case then I will likely just fork my own version in order to add this feature

    Also migrating this post to github for further development:
    https://github.com/perrybutler/WP-OAuth/issues/52

    Thread Starter Xtremefaith

    (@xtremefaith)

    It looks like if I just change login-google.php line:

    define('SCOPE', 'profile');

    to:

    define('SCOPE', 'https://www.googleapis.com/auth/plus.profile.emails.read');

    Now the plan is to use this in the wp-oauth.php file at the wpoa_login_user() process just after wpoa_match_wordpress_user(). How does this sound to you. Is this something that would not work for the direction of your plugin (if so I may need to just fork this and make my own custom updates).

    Please let me know your thoughts.

    Thread Starter Xtremefaith

    (@xtremefaith)

    Within get_oauth_identity() (in login-google.php) I added a scope parameter and set it to 'email', I also tried https://www.googleapis.com/auth/plus.profile.emails.read.

    Unfortunately neither of these work. I don’t understand why not from what I’ve read of Google’s documentation. While playing on the API Explorer it returned as expected but only when accessing my own account. This leads me to believe that when attempting through plugin perhaps it lacks permissions even though it was granted.

    I will keep attempting this because I need it for a couple sites both with Google and Windows, so any help would be really appreciated. Depending on what needs updated I may work directly with you to contribute changes so please let me know.

    Thread Starter Xtremefaith

    (@xtremefaith)

    I’m pretty sure I could write a patch into this IF I can figure out where the email that was used is stored. When I debug the $oauth_identity I only get the provider and ID. And if I debug the $result_obj I get the following keys:

    kind, etag, gender, objectType, id, displayName, name, familyName, givenName, url, image, url, isDefault, isPlusUser, language, circledByCount, verified

    Is there not a way to get the email?

    Same issue. When I deactivate either Yoast or BuddyPress it goes away (but I think that’s just simply because it’s a BuddyPress function that Yoast is using incorrectly). Anyone have progress on this? Not interested in turning off debug while in development, want to resolve the issue.

    Thread Starter Xtremefaith

    (@xtremefaith)

    In case it helps, I came across this:
    https://wordpress.stackexchange.com/questions/90065/preserve-white-space-in-page

    I noticed that in some cases the white spaces are not preserved and I think its because WordPress editor is expecting <pre> & <code> in the html tab and not backticks.

    Trolling: make a deliberately offensive or provocative online posting with the aim of upsetting someone or eliciting an angry response from them.

    “yes, exist addon (29$!!!) but why pay for this if other plugin make this as free? is no sense and is nosense a plugin manager without a page.
    i’ve lost some hours of my life to install, try and found that not exist a download page!”

    Sounds like trolling with really bad english, who compares himself to another troll and says I’m not as bad as him (welcome to humanity’s view of morality).

    Just to be clear to anyone reading this review, I have no connection to the author of this plugin, I simply use it with a national franchise for 4+ years now without any issues. Salvatore’s review is a feature request at best and should not have a barring on the quality of plugin that authors have made up until this point.

    So the plugin works fine the way it was intended but since it doesn’t do what you want it to do you are trolling the author who put some legitimate effort into making this work for 70,000+ people for free?

    Not sure you understand what the review process is for but would advise you post your questions or concerns to the developer in the forums where they can properly deal with it and leave an honest review here (if at all).

    Many people do not need the feature you are requesting, and there is nothing wrong with a developer creating a premium feature (perhaps you suggest it as a feature to be added in a future release). If you are willing to endorse the “free” plugin you mentioned than use that, but the fact that it exist should not reflect poorly on this author’s craftsmanship

    I understand your frustrations but hope you can respect my comments

    I’ve been attempting the same thing but correct me if I’m wrong, doesn’t this still lead the user to the WP-login.php page of they:

    • Attempt the wrong username/password
    • Forget password

    I think we need a way to only use WP-login.php (or other) for POST and not as an actual page

    Glad you guys started this post cause I was about to do the same. Is it possible load on only selected pages? With that addition I’m willing to vote 5 stars simply for how easy it was to setup ??

    Xtremefaith

    (@xtremefaith)

    If its any help to anyone. I had this problem as well because as with most host, the only way to direct the primary domain to a directory other than root is to use a .htaccess method, and that works fine except when going to /wp-admin (no trailing “/”) because when it redirects to the wp-login.php it adds a redirect_to parameter in the URL, which in my case meant it included the “_production” directory in the absolute path.

    I think this is a bug with WP, but either way the way to fix it was to insert this line in the _production directory .htaccess file.

    # BEGIN WordPress
    <IfModule mod_rewrite.c>
    RewriteEngine On
    RewriteBase /
    RewriteRule ^index\.php$ - [L]
    RewriteRule ^([_0-9a-zA-Z-]+/)?wp-admin$ $1wp-admin/ [R=301,L]  <-- This line
    RewriteCond %{REQUEST_FILENAME} !-f
    RewriteCond %{REQUEST_FILENAME} !-d
    RewriteRule . /index.php [L]
    </IfModule>
    
    # END WordPress

    Hope that helps

    Same here, please don’t say something like set WP_DEBUG to false. ??

    I see you figured out your answer but the bug still remains from what I can see. When I add custom post type relationship field to an options page right now I’m getting that same notice:

    Notice: Trying to get property of non-object in \acf\core\fields\relationship.php on line 496

    Any thoughts on how this should resolved?

    If you can clearify you question I may be able to help. Currently my custom starter theme allows you to set options in option pages that then effect ACF fields used throughout the site.

    You must export the fields to PHP so they are initiated in the code not the database. Then from that point its all about where you hook. Since 4.3 you can use get_field() within your functions.php scope

    4.3.0
    Core: get_field can now be used within the functions.php file

    I believe what you may be attempting is possible but I need better details to give you specifics. ACF is an awesome plugin and I’ve used it to great depths that I doubt there is anything it can’t do ??

    Xtremefaith

    (@xtremefaith)

    You are not alone. Are you getting image that are distorted but readable? (lines through image, weird coloring etc)? Hopefully someone can verify if it is a WP issue of host, at the moment I say WP because I build a lot of sites on different host and have only experience this on 3.7+ thus far.

Viewing 15 replies - 1 through 15 (of 26 total)