Fixed report/block user hidden for valid WordPress.com authors#22594
Fixed report/block user hidden for valid WordPress.com authors#22594annaharri89 wants to merge 2 commits intowordpress-mobile:trunkfrom
Conversation
…sSelfHosted check used ‘authorBlogId’ (author's site ID) which is -1 for authors without their own site but didn’t account for self-hosted Jetpack blogs connected to WordPress.com. Replaced isSelfHosted with ‘authorId > 0’ to match what the report/block actions actually require, which is that the user has a word press account.
|
jkmassel
left a comment
There was a problem hiding this comment.
Thanks for the contribution @annaharri89!
Reviewing this, I'm still not sure it's the correct approach – I think all we need to do here is check that the user is currently logged into WordPress.com in order to show that button – if they are, they should be able to block a feed, site, or user that way.
Happy to re-review or discuss further!
|
@jkmassel I thought you need the id of the user you want to report or block. Even if the current user is signed in to wordpress, that doesn't guarantee a post's author has a wordpress account does it? Or am I misunderstanding? (hasWordPressAccount is about the post's author not the signed in user) |



Description
Fixes #21144
isSelfHosted()from ReaderUtils and its wrapper.Testing instructions
Before my fix:

After my fix:
