current community

your communities

more stack exchange communities

company blog
Stack Exchange Inbox Reputation and Badges
tour help
Join the Stack Overflow Community
Stack Overflow is a community of 6.3 million programmers, just like you, helping each other.
Join them; it only takes a minute:
Sign up
up vote 2 down vote favorite
1

recently I have gotten this error:

Parse error: syntax error, unexpected ')' in /home/aps/public_html/wp-content/themes/Valor/functions.php on line 54

I have looked at this line and found no problems, and ran it through a PHP error code finder.

Just to be sure, I copied the code from the functions.php I have saved on my computer into the editor and it still happens.

I am using wordpress and the Valor theme.

I had been editing these lines when it happened:

if ( function_exists( 'add_theme_support' ) ) { // Added in 2.9
    add_theme_support( 'post-thumbnails' );
    add_image_size( 'game_feature', 1000, 500, true );
    add_image_size( 'story_feature', 150, 100, true );
    add_image_size( 'video_feature', 150, 200, true );
    add_image_size( 'review_thumb', 120, 80, true );
    add_image_size( 'post_image', 660, 350, true );

I changed all the 'true' to 'false', because I wanted the thumbnails on my site to fit into the box, rather than be cropped. This didn't do anything so instead I just took away 'true' and 'false' entirely, leaving a blank space.

It was after I did this and hit 'save' that the error came up. I quickly went back and changed them all to 'true' in order to undo the error, but from what you can see and from what I have already told you, it didn't work, despite my efforts.

I'm very new to PHP, and the only explanation I can think of is that by deleting those values it caused another .php file to change something, but like I said, I'm new.

Line 54 is: add_image_size( 'game_feature', 1000, 500, true );

share|improve this question

closed as off-topic by andrewsi, madth3, Shankar Damodaran, Soner Gönül, NDM Sep 27 '13 at 7:40

This question appears to be off-topic. The users who voted to close gave this specific reason:

  • "Questions concerning problems with code you've written must describe the specific problem — and include valid code to reproduce it — in the question itself. See SSCCE.org for guidance." – andrewsi, madth3, Shankar Damodaran, Soner Gönül
If this question can be reworded to fit the rules in the help center, please edit the question.
    
did you try again reloading your file as this code seems to be fine. – Sunil Verma Sep 26 '13 at 12:53
1  
Your code is fine. Can you look into other parts of the file? – Pupil Sep 26 '13 at 12:54
    
Sorry I forgot to specify, I know that the code is fine. I ran it through a few checks and even copied the original code from the file on my computer onto it, and still no luck. – Zak Parker Sep 26 '13 at 12:57
    
Here is the code: pastebin.com/e7yDK9gx I had to put it in pastebin because it was too long to fit in here. – Zak Parker Sep 26 '13 at 13:01
    
on line 13 you have an if statement with no {}. – Jim Sep 26 '13 at 13:04
up vote 0 down vote

if you get tired of trying to find/solve the problem (as your code is fine), start from scratch ie active another theme, then delete valor theme, paste unmodified valor theme back, then activate valor theme.

fyi, revision control might help the next time you make a mistake, ctrl+z is good, until you close the file. mistakes are the only way you learn.

share|improve this answer
    
Yeah I might try that, thanks. – Zak Parker Sep 26 '13 at 13:13
up vote 0 down vote

On line 13, you have:

if ( function_exists('register_sidebar') )

You forgot to put the opening quote at the end - { The line 13 should look like:

if ( function_exists('register_sidebar') ) {

EDIT:
Don't forget to close the tag on line 31. }

EDIT 2:
Try the solution that @DimitriMostrey said in the comments it might work.

change true to 1 and false to 0, the only possiblity i can see is that you made a function called true and that your scripts does expect a '(' Even if the number of arguments in the add_image_size function are not correct, it would not call a parse error.

share|improve this answer
    
Thanks, but it's still not working. – Zak Parker Sep 26 '13 at 13:07
    
Just to be sure, I made it look like this now: /* SIDEBARS */ if ( function_exists('register_sidebar') ) { register_sidebar(array( 'name' => 'Sidebar', 'before_widget' => '<li class="sidebox %2$s">', 'after_widget' => '</li>', 'before_title' => '<h3 class="sidetitl">', 'after_title' => '</h3>', )); register_sidebar(array( 'name' => 'Footer', 'before_widget' => '<li class="botwid %2$s">', 'after_widget' => '</li>', 'before_title' => '<h3 class="bothead">', 'after_title' => '</h3>', )); } – Zak Parker Sep 26 '13 at 13:07
    
I cannot see Dimitris comment. What does it say? – Zak Parker Sep 26 '13 at 13:12
    
I put it as a quote in my question. – VladHQ Sep 26 '13 at 13:13

Not the answer you're looking for? Browse other questions tagged or ask your own question.