Hi,
I've used the previous versions of your theme without any trouble.
I was glad to find a new release and downloaded it right away. I also deleted the previous files since I was not going to use them.
Started a clean install and when I chose simply modern as default theme I got Parse error: parse error in C:\wamp\www\politologia\sites\all\Themes\simply_modern\page.tpl.php on line 391...
Downloaded it many times just in case it was a problem with my download, but I get the same.
I don't know much about this things, is there something not closed in the code or is it just a personal problem?
Thanks for your response...
Comment | File | Size | Author |
---|---|---|---|
#5 | simply_modern.patch.txt | 795 bytes | Takafumi |
Comments
Comment #1
plastique CreditAttribution: plastique commented1.3 works just fine on my site, looking for any differences...
Comment #2
afaaro CreditAttribution: afaaro commentedI have the same error:
Parse error: parse error in C:\wamp\www\drupal\sites\all\Themes\simply_modern\page.tpl.php on line 391
Comment #3
toomanypets CreditAttribution: toomanypets commentedI have the same problem:
Parse error: parse error in C:\WampServer\www\drupal_6\sites\all\themes\simply_modern\page.tpl.php on line 391
Comment #4
andrewmacpherson CreditAttribution: andrewmacpherson commentedI'm not seeing this error. I've been using 6.x-1.4 for a few weeks now.
I had a look at page.tpl.php and guess what? There is no line 391 - the file is 390 lines long.
Comment #5
Takafumi CreditAttribution: Takafumi commentedThe cause of this issue is use of php's short tag, so this patch fixes it.
Comment #6
martinwjones CreditAttribution: martinwjones commentedI'm sorry @Takafumi but this patch appears to be implemented in the code I am running (downloaded today) and I still get the same errors as everybody else has reported above:
Parse error: parse error in {mypath}\themes\simply_modern\page.tpl.php on line 391
I have installed the Themes Settings API - nothing seems out of place.
Cheers
Martin
Comment #7
Takafumi CreditAttribution: Takafumi commented@martinwjones: See line 146 on page.tpl.php. You should find '<? } ?>' there.
Comment #8
apadernoThe Theme Settings API functionallity is included in the latest Drupal 6 versions; you should install the module if you have an old Drupal 6 version.
The module you installed is not even compatible with Drupal 6.3 or higher.
Comment #9
martinwjones CreditAttribution: martinwjones commented@Takafumi - apologies, you are correct (although a little cryptic through brevity).
I changed the '<? } ?> to '
}
on line 146 and it seems to be fixed.Cheers
Martin
Comment #10
plastique CreditAttribution: plastique commentedThank you, works perfectly!
Comment #11
jrglasgow CreditAttribution: jrglasgow commentedthe patch on #5 has been committed