Cannot use isset() on the result of a function call - Laravel

Question

I have a custom helper and a config, I can get the value but when I try the isset() it's not working and it shows me like this

and this is the content of customer helper: app\helper\LmsHelper.php

class LmsHelper
{   
    private static $host;
    private static $api_key;
    private static $api_version;
    private static $use_ssl;
    private static $debug;
    private static $port;

    public function __construct() {
        self::$host         = Config::get('lms.host');
        self::$api_key      = Config::get('lms.api_key');
        self::$api_version  = Config::get('lms.api_version');
        self::$use_ssl      = Config::get('lms.use_ssl');
        self::$debug        = isset(Config::get('lms.debug')) ? Config::get('lms.debug') : false;


        if (isset(Config::get('lms.port'))) {
            self::$port = Config::get('lms.port');
        } else {
            self::$port = self::$use_ssl ? 443 : 80;
        }
    }
}

and app\config\lms.php

<?php

return array( 
    'host' => 'www.themartialarts.university', 
    'api_key' => '6027-178512-e272b9afac4763199fa8d79e4bc0dcb39a378658', 
    'api_version' => '1', 
    'use_ssl' => true,
);

any idea what other method I can do? cause this is from normal PHP and I convert it into Laravel.

I think you should not use the "isset" function, as explained here stackoverflow.com/questions/11464714/php-check-if-false-or-null — ddb
– ddb, Commented Jul 4, 2016 at 8:31

Philip Rollins · Accepted Answer · 2016-07-04 08:31:42Z

3

Use the Config::has("name") method call since it's a method call you can't use isset().

answered Jul 4, 2016 at 8:31

Philip Rollins

1,2899 silver badges20 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Niclas Larsson · Accepted Answer · 2016-07-04 08:36:15Z

1

You can only call isset on $variables.

Change that line to self::$debug = Config::get('lms.debug', false); to solve the problem. (set the default value to false)

edited Jul 4, 2016 at 8:36

answered Jul 4, 2016 at 8:30

Niclas Larsson

1,3178 silver badges13 bronze badges

Comments

Álvaro González · Accepted Answer · 2016-07-04 08:33:21Z

0

Even if the language construct supported it, it doesn't make much sense to test whether the function returns a variable because even missing return clauses generate a proper null. E.g.:

function nothing() {
}

var_dump( nothing() );

NULL

I suspect you want to use is_null() rather than isset().

answered Jul 4, 2016 at 8:33

Álvaro González

147k45 gold badges282 silver badges378 bronze badges

Comments

Ravi Hirani · Accepted Answer · 2016-07-04 08:34:56Z

0

Use if instead of isset as isset is used to check a variable is set and is not NULL.

answered Jul 4, 2016 at 8:34

Ravi Hirani

6,5341 gold badge31 silver badges43 bronze badges

Comments

score 0 · Accepted Answer · 2016-07-04 08:38:15Z

0

The " isset " takes only variable. You must set the result of your function in a prior varriable.

 public function __construct() {
    $config = Config::get('lms.debug');
    self::$host         = Config::get('lms.host');
    self::$api_key      = Config::get('lms.api_key');
    self::$api_version  = Config::get('lms.api_version');
    self::$use_ssl      = Config::get('lms.use_ssl');
    self::$debug        = isset($config) ? Config::get('lms.debug') : false;

    $configPort = Config::get('lms.port');
    if (isset($configPort)) {
        self::$port = Config::get('lms.port');
    } else {
        self::$port = self::$use_ssl ? 443 : 80;
    }
}

edited Jul 4, 2016 at 8:38

answered Jul 4, 2016 at 8:32

user6029681

1 Comment

Niclas Larsson Over a year ago

What about if (isset(Config::get('lms.port'))) {? :P

Collectives™ on Stack Overflow

Cannot use isset() on the result of a function call - Laravel

5 Answers 5

Comments

Comments

Comments

Comments

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

Comments

Comments

Comments

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related