-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rename Blog.isHostedAtWPcom for clarity #15089
Comments
I've also noticed WordPress.com sites with plugins installed can't be switched to private using WPiOS but can be switched using WPAndroid. This happens because of: WordPress-iOS/WordPress/Classes/Models/Blog.m Lines 504 to 506 in 8bfca46
Tested on WPiOS 15.8 and WPAndroid 15.9 (beta) |
WPiOS doesn't show a Language option on the Settings screen (I'd expect to see it under the GENERAL section, below Time Zone) for WordPress.com sites with plugins installed. Based on the following comment though, I think the fact that the Language option is not present might be an oversight: WordPress-iOS/WordPress/Classes/ViewRelated/Blog/Site Settings/SiteSettingsViewController.m Lines 237 to 240 in 50e1647
|
This issue has been marked as stale because:
Please comment with an update if you believe this issue is still valid or if it can be closed. This issue will also be reviewed for validity and priority during regularly scheduled triage sessions. |
Expected behavior
By its name alone, one would expect
Blog.isHostedAtWPcom
to betrue
for all WordPress.com sites, whether or not they have plugins installed. It's nottrue
in this scenario, because it's actually just an indication of whether or not Jetpack is installed. So it would be better to rename it to make this clearer.Actual behavior
Blog.isHostedAtWPcom
isfalse
for WordPress.com sites with plugins installed because it's actually just the negation of!blog.jetpack
.The current naming is likely the reason @-mentions is disabled on WordPress.com sites with plugins installed. So there are likely real negative consequences to keeping the current naming.
Steps to reproduce the behavior
isHostedOnWPcom
isfalse
/0
:WordPress-iOS/WordPress/Classes/Services/BlogService.m
Line 723 in b347ef8
Proposed solution
I think we should rename it to
Blog.isWPComSimpleSite
. This would involve renaming the variable in code as well as in the Core Data data model.Tested on iPhone, iOS 14, WPiOS 6895ab8
The text was updated successfully, but these errors were encountered: