<div dir="ltr">So all this comes down to what we were saying in this discussion all along<div>or at least from the second Theme submission, that there are no GPL issues. </div><div><br></div><div>Am I wrong Justin?</div><div>
<br></div><div>If so we already touched the policy and noted that this should be acceptable.</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Sep 20, 2013 at 3:14 PM, Justin Tadlock <span dir="ltr"><<a href="mailto:justin@justintadlock.com" target="_blank">justin@justintadlock.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The issue here is really all about whether a theme can block features/code bundled within the theme itself without a paid subscription. It's not connecting to an API to use a service (like Akismet). It's connecting to an API to determine whether particular code in the theme will run based on nothing more than a payment.<br>
<br>
For a simple example, this code in `footer.php`:<br>
<br>
<?php if(!pl_is_pro()):?><br>
<a class="pl-credit" href="<a href="http://www.pagelines.com/" target="_blank">http://www.pagelines.<u></u>com/</a>" title="Built with PageLines DMS [basic]" target="_blank" style="display: block !important; visibility: visible !important; opacity: 1 !important;"><br>
<i class="icon-pagelines pl-transit"></i> <span class="fademein">DMS</span><br>
</a><br>
<?php endif; ?><br>
<br>
I'm sure most of you can understand it, but I'll break it down:<br>
<br>
* A footer link is displayed if you do not have a subscription to their pro service.<br>
* The inline style rules break our guidelines on inline style rules.<br>
* The inline style rules seem to be there solely to make it hard for users to hide this via CSS.<br>
<br>
Obviously, anyone can remove that if they know where to look and a little bit about code. But, the lengths in which the theme author has gone to block users from removing a footer credit wouldn't be allowed in any other theme.<br>
<br>
I haven't had a chance to fully look over the code, but I'm guessing based on the comments so far, that this is just one of many similar restrictions. It seems the gist of everyone's concern is about actual restriction within the theme code that's being put on WordPress.org.<br>
<br>
Let's all assume that there's no GPL issues. The question then becomes whether this violates WordPress.org policy and/or theme review policy.<br>
<br>
So, let's talk policy.<br>
<br>
Assuming I understand the issues clearly, let me present another example that's not tied to the theme in question.<br>
<br>
Suppose in the next theme I upload to WordPress.org, I decided to create some color and layout options for the theme customizer. This code would, obviously, be within the theme. However, anyone who downloads this theme from WordPress.org would not be allowed to use my theme customizer options out of the box. They'd need to purchase an API key for this feature to become unlocked. Of course, anyone who knows a little PHP can go in and change the code so that it works without an API key.<br>
<br>
>From what I've gathered from this discussion, that example is no different (please do correct me if I'm wrong). I'm not sure if that necessarily breaks the GPL (probably not). It's more a matter of policy -- whether this is something that we should allow on the WordPress.org theme repository.<br>
<br>
I apologize if I'm misunderstanding anything that's happening with the code and hope I didn't misrepresent the situation. I'll be sure to give it a better look when I get a chance.<div class="HOEnZb"><div class="h5">
<br>
<br>
On 9/20/2013 2:50 PM, Otto wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Fri, Sep 20, 2013 at 2:40 PM, Bryan Hadaway <<a href="mailto:bhadaway@gmail.com" target="_blank">bhadaway@gmail.com</a>> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Because, I'm going to bang on pots and pans until you actually acknowledge<br>
that the horse ever even existed. The issue isn't contained within the walls<br>
of the theme's code, it's a licensing issue that has not changed just<br>
because a new version of the theme was uploaded.<br>
</blockquote>
In my view, the issue existed in the theme, up until they moved the<br>
problem code to a plugin on their own site. And I'm kind of okay with<br>
that.<br>
<br>
So as far as I'm concerned, the issue is not an issue anymore.<br>
<br>
Now, as I see it, you're wanting to create a whole new issue, one<br>
where now we impose even more strict guidelines on theme authors. As<br>
near as I can tell, you essentially want the theme review team to say<br>
that subscription models are unacceptable. Well, call me crazy, but I<br>
don't think that's a really good idea.<br>
<br>
If you want to fine-pick the nitty gritty details and go "oh-noes<br>
they're selling code that actually checks for subscriptions", well I'm<br>
going to say that checking for subscriptions really doesn't bother me<br>
too awful much, as long as the code to do so ain't on WordPress.org.<br>
<br>
So yeah. That's my viewpoint. I don't much care whether they have a<br>
subscription model or not for their own code that is being sold from<br>
their own site. My 2 cents.<br>
<br>
-Otto<br>
______________________________<u></u>_________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.<u></u>wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/<u></u>mailman/listinfo/theme-<u></u>reviewers</a><br>
</blockquote>
<br>
______________________________<u></u>_________________<br>
theme-reviewers mailing list<br>
<a href="mailto:theme-reviewers@lists.wordpress.org" target="_blank">theme-reviewers@lists.<u></u>wordpress.org</a><br>
<a href="http://lists.wordpress.org/mailman/listinfo/theme-reviewers" target="_blank">http://lists.wordpress.org/<u></u>mailman/listinfo/theme-<u></u>reviewers</a><br>
</div></div></blockquote></div><br></div>