Comments (7)
This has nothing to do with theme review. I don't think it belongs here.
from wpthemereview.
How come? WooCommerce templates are directly related to themes...
from wpthemereview.
The versioning of anything in the theme has nothing to do with theme review, except perhaps TGMPA itself.
It's not a part of review to determine the if the version of Bootstrap or Font Awesome or Woocommerce templates are the latest. Perhaps the intent is to support legacy installations?
This is out of scope.
from wpthemereview.
I have to say I agree with @joyously on this one.
If this is something that folks want added, it should probably be under the theme review guidelines first.
from wpthemereview.
I would agree that it should be under the theme review guidelines. If a theme is including code in relation to other plugins, it probably should be part of the review process to ensure with those plugins active there aren't fatals or notices. I've encountered several times an outdated template in woocommerce results in php fatals/notices on frontend.
I don't agree that it's out of scope though, something like bootstrap and fontawesome, yeah those would be, but woocommerce directly deals with the frontend presentation, requires php code to be added in WordPress themes, and goes untested during reviews. Keeping those template files up to date for overrides is just one of the ways that it could at help theme authors are submitting quality themes from the beginning.
At some point the buddypress tag was added for themes, so someone else felt the same way about something that is so intrusive to the frontend of a site/theme. Given that buddypress has 200k+ installs compared to woocommerce at 4m+ installs, I think it deserves to have something in place to reduce chances of errors.
Even though these standards are used for the WordPress review process, many people will use these standards outside of .org for places like envato, or even just client sites. It's just one additional way to increase the quality of themes being delivered to end users who often times don't know what to do when they get fatals or notices.
Plus - the theme checks are already scanning woocommerce templates folder - so if the decision is to not have anything to do with woocommerce and not adding these checks, the review process and theme -sniffer should then skip this folder for things like globals checks etc that they common have in their templates. The process is saying you have to conform to the theme review standards, but we don't care if the end result is that user's sites don't work because there's a template that causes fatals now.
from wpthemereview.
I would rather see a warning for non-core post types or something. It seems to me that if a plugin creates a post type, it should provide the output of it (template). We should be encouraging theme authors to provide easy ways to handle any post type instead of providing templates for a specific post type from a popular plugin.
I can see a theme overriding CSS classes for woocommerce, but I think template files belong in the plugin.
from wpthemereview.
I'll mark this as wontfix as it is kinda out of the scope for the TRT (at the moment). If the need arises, we can reopen this one.
from wpthemereview.
Related Issues (20)
- variables in template files are flagged as global HOT 15
- Exceptions to prefixing hook names HOT 7
- False positive on loading stylesheet in JS file HOT 4
- Dealing with rules that shouldn't apply to templates HOT 1
- Add sniff: WordPress.PHP.IniSet HOT 6
- [New sniff] Check if page templates are using reserved prefix HOT 1
- Detect WPTRT feature project versions HOT 3
- Add sniff documentation HOT 1
- Minimal PHP version HOT 11
- Whitelist wp_body_open function from PrefixAllGlobals sniff HOT 5
- sniff: plugin territory - action hook wp_edit_nav_menu_walker
- File name with reserved slug in subfolder HOT 11
- Optimisation and minor grammar fixes
- [Implement sniff] Warn about short ternary usage in themes HOT 9
- Internal.Exception -- function update_current_version() should be removed.
- [Update existing sniff] Allow register_block_style HOT 1
- Allow `add_menu_page` & `add_submenu_page` HOT 9
- twentyseventeen theme: posts with no date - but date is shown in similar posts HOT 1
- 👍
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from wpthemereview.