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...

CommentFileSizeAuthor
#5 simply_modern.patch.txt795 bytesTakafumi
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

plastique’s picture

1.3 works just fine on my site, looking for any differences...

afaaro’s picture

I have the same error:

Parse error: parse error in C:\wamp\www\drupal\sites\all\Themes\simply_modern\page.tpl.php on line 391

toomanypets’s picture

I 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

andrewmacpherson’s picture

I'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.

Takafumi’s picture

Priority: Minor » Normal
Status: Active » Needs review
FileSize
795 bytes

The cause of this issue is use of php's short tag, so this patch fixes it.

martinwjones’s picture

I'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

Takafumi’s picture

@martinwjones: See line 146 on page.tpl.php. You should find '<? } ?>' there.

apaderno’s picture

I have installed the Themes Settings API - nothing seems out of place.

The 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.

martinwjones’s picture

@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

plastique’s picture

Thank you, works perfectly!

jrglasgow’s picture

Status: Needs review » Fixed

the patch on #5 has been committed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.