madalinungureanu / user-profile-picture

WordPress User Profile Picture
GNU General Public License v2.0
15 stars 7 forks source link

Adding language files (Polish) and repair editing own avatar function #1

Closed Monter closed 8 years ago

ronalfy commented 8 years ago

@Monter is the sole purpose of the plugin changes to add the title attribute to the image? Just clarifying.

Monter commented 8 years ago

No. Please start the plug-in, add your own avatar and see what happens when you want to click on it to change. Link not only does not have a title, but also directs the wrong place (yourblog.domain/?mt_pp= [id]). This is caused by an erroneous construction of the link to the picture, my amendment improves it.

ronalfy commented 8 years ago

@Monter thanks, that makes sense. Could you paste the HTML you're getting? I'm not seeing the link you are mentioning.

Could something be inadvertently hooking into get_the_post_thumbnail and adding the extra HTML you mentioned?

Monter commented 8 years ago

Fixed.

Monter commented 8 years ago

This is HTML code after adding my own picture (before changes in this pull request): wrong_link After changes everything work O.K.

ronalfy commented 8 years ago

Ah, yeah, something is definitely hooking into that filter and inserting a link around the image. I'll push out an update tonight if I have time. Thanks for the PR.