ErrorException [ Parse Error ]: syntax error, unexpected T_ENDIF

Question

I want to validate or show the radio button checked,if some value is present in database. Please refer my views code,

<input type="radio" name="bothadvandlor"  value="1" <?php if($advs->is_checked == "1");?>checked="checked"<?php endif;?> class="advs"/> Show Google Ads<br/>
<input type="radio" name="bothadvandlor"  value="2" <?php if($advs->is_checked == "2");?>checked="checked"<?php endif;?> class="advs"/> Others

The above is the form i tried,but i am getting the syntax error message like "ErrorException [ Parse Error ]: syntax error, unexpected T_ENDIF" .

Need help to solve this.

score 3 · Answer 1 · answered Mar 10 '14 at 10:13

3

Should be colon :,

if($advs->is_checked == "1"):
                            ^

Reference.

answered Mar 10 '14 at 10:13

Rikesh

26,156
14
79
87

score 2 · Accepted Answer · answered Mar 10 '14 at 10:13

You need to use colon : instead of semi-colon ; here after your if condition:

if($advs->is_checked == "1"):
// -----------------------  ^ here

if($advs->is_checked == "2"):
// -----------------------  ^ and here

From the docs:

PHP offers an alternative syntax for some of its control structures; namely, if, while, for, foreach, and switch. In each case, the basic form of the alternate syntax is to change the opening brace to a colon (:) and the closing brace to endif;, endwhile;, endfor;, endforeach;, or endswitch;, respectively.

score 2 · Answer 3 · answered Mar 10 '14 at 10:14

<input type="radio" name="bothadvandlor"  value="1" <?php if($advs->is_checked == "1"):?>checked="checked"<?php endif;?> class="advs"/> Show Google Ads<br/>
<input type="radio" name="bothadvandlor"  value="2" <?php if($advs->is_checked == "2"):?>checked="checked"<?php endif;?> class="advs"/> Others

score 1 · Answer 4 · answered Mar 14 '14 at 11:03

I just wanted to add that Kohana has got some neat stuff for this:

public static radio( string $name [, string $value = NULL , boolean $checked = bool FALSE , array $attributes = NULL ] )

e.g.

<?php echo Form::radio('bothadvandlor', 1, ($advs->is_checked == '1'), array('class' => 'advs')); ?> Show Google Ads <br />

I think it makes these kinds of operations a whole lot easier. More separation of HTML and PHP thus better readability.

ErrorException [ Parse Error ]: syntax error, unexpected T_ENDIF

4 Answers4