[theme-reviewers] Problem uploading new version of Hybrid theme

[Edward Caissie]

Nice find Dave;

I looked in every file except the ones with image extensions.


Cais.

On Tue, Jun 22, 2010 at 2:29 PM, Dave Jesch <davejesch at gmail.com> wrote:

> I found a short tag in the screenshot.png file. Two of them, in fact.
>
> My guess is that the automated process that looks for the short tags looks
> in every file. It should, since some programmers like to put PHP code in a
> .inc or other non-standard file. But I think certain file types can be
> safely excluded from the search, like image files, .txt and .css and maybe a
> few others.
>
> Dave
>
>
> On Tue, Jun 22, 2010 at 6:22 AM, Edward Caissie <edward.caissie at gmail.com>wrote:
>
>> I just did a similar search using one of my editors and was unable to find
>> a "short tag" instance ... is this a known issue? or, is there simply
>> something with the sheer number of files in the theme? (*grin*)
>>
>>
>> Cais.
>>
>> On Tue, Jun 22, 2010 at 3:34 AM, <justin at justintadlock.com> wrote:
>>
>>> Hi,
>>>
>>> The theme uploader gives me this error message:
>>>
>>> "Your theme is using PHP short tags - <? - instead of normal PHP tags -
>>> <?php - in some places. Please only use normal PHP tags in your themes, as
>>> some sites have short tags disabled. If you have questions about this please
>>> contact theme-reviewers at lists.wordpress.org<//theme-reviewers at lists.wordpress.org>
>>> ."
>>>
>>> Assuming it was correct, I done a search in all theme files (using
>>> Notepad++) for the offending tag(s).  The search turned up no results.
>>>
>>> There may be an error with the uploader for some reason.  Can you please
>>> look into this or add version 0.8 of my theme (attached) to the theme
>>> repository?
>>>
>>> Justin Tadlock
>>> justin at justintadlock.com <//justin at justintadlock.com>
>>>
>>> _______________________________________________
>>> theme-reviewers mailing list
>>>
>>> theme-reviewers at lists.wordpress.org
>>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>>
>>>
>>
>> _______________________________________________
>> theme-reviewers mailing list
>> theme-reviewers at lists.wordpress.org
>> http://lists.wordpress.org/mailman/listinfo/theme-reviewers
>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.wordpress.org/mailman/private/theme-reviewers/attachments/20100622/249eeaea/attachment.htm>