Right syntax in an "isset(...) ? :" scheme

Question

I have this line

$logos = isset($instance['client_logo']) ? $instance['client_logo'] : [''];

causing this

Parse error: syntax error, unexpected '[' in .../widget-home-clients.php on line 18

How does it need to be changed?

`$logos = isset($instance['client_logo']) ? $instance['client_logo'] : '';` — Daan, Apr 07 '16 at 11:02

score 3 · Answer 1 · edited Apr 07 '16 at 11:15

3

If [''] is intented to give you an array holding one empty string, you should do array(''), because it appears your PHP version doesn't yet allow for this short syntax - it was introduced with PHP 5.4.

If you just wanted to get an empty string, just replace [''] with ''

edited Apr 07 '16 at 11:15

Narf

14,600
3
37
66

answered Apr 07 '16 at 11:10

nl-x

11,762
7
33
61

score 0 · Answer 2 · answered Apr 07 '16 at 11:02

0

Use this:

$logos = (isset($instance['client_logo'])) ? $instance['client_logo'] : '';

answered Apr 07 '16 at 11:02

Md Mahfuzur Rahman

2,319
2
18
28

4

Why add the extra `()` ? – Daan Apr 07 '16 at 11:02
`()` is not mandatory. – Md Mahfuzur Rahman Apr 07 '16 at 11:05
I can also add 100 times `()` around `isset()` but why ? – Daan Apr 07 '16 at 11:06
1

No special reason. My style. – Md Mahfuzur Rahman Apr 07 '16 at 11:15

score 0 · Answer 3 · answered Apr 07 '16 at 11:26

0

Replace your code with the following code...

<?php 
$logos = '';
if(isset($instance['client_logo']) && !empty($instance['client_logo']))
{
    $logos= $instance['client_logo'];
}
?>

answered Apr 07 '16 at 11:26

Manjeet Barnala

2,975
1
10
20

Right syntax in an "isset(...) ? :" scheme

3 Answers3