php passing arguments in constructor

Question

Can anyone tell me why $Role is being ignored?

I am trying to pass an argument and it is always getting a null value, however, when I call the method the var_dump shows that the $Role is 2.

When I use the var_dump inside getListFromDB the $Role is being set to null.

Method getListFromDB()

function getListFromDB($tableName, $orderBy = 'Description', $where = null, $Role = null) {
    DO_Common::debugLevel(0);
    if (empty($tableName) || empty($orderBy))
        throw new Exception("tableName and orderBy cannot be left empty");

    var_dump($Role);

    if (!empty($Role))
    {
        echo "here";
        if ($Role === 2)
        {
            if ($tableName == 'AssetTypes')
            {
            $params = array('tableName' => 'AssetTypes',
                            'orderBy' => $orderBy,
                            'whereAdd' => 'Restricted = 1');
            }

            var_dump($params);

        }
        else
        {
            $params = array('tableName' => $tableName,
                    'orderBy' => $orderBy);           
           var_dump($params);
        }        
    }
    else
    {
         $params = array('tableName' => $tableName,
                    'orderBy' => $orderBy);
         //var_dump($params);
    }

    if (!empty($where) && $table != 'AssetTypes') {
        if (strpos(strtolower($where), 'flag') === false)
            $where .= " AND Flag != " . fDELETED;

        $params += array('whereAdd' => $where);
    }

    return DO_Common::toAssocArray($params);
}

How the method is being called:

$AssetTypesOptions = getListFromDB('AssetTypes', $Role);

Is there something I am missing here?

AbraCadaver · Accepted Answer · 2015-10-29 15:09:28Z

1

$Role is the fourth argument for the function but you are sending it as the second argument:

$AssetTypesOptions = getListFromDB('AssetTypes', 'Description', null, $Role);

answered Oct 29, 2015 at 15:09

AbraCadaver

79.2k7 gold badges75 silver badges91 bronze badges

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

1 Comment

Brian Over a year ago

Wow! Completely missed it! Thanks!

Jan · Accepted Answer · 2015-10-29 15:09:23Z

1

You should call it as the fourth argument, like so:

getListFromDB("tablename", "some fancy description", "here", $Role);

I made them up obviously...

answered Oct 29, 2015 at 15:09

Jan

43.3k11 gold badges57 silver badges87 bronze badges

Collectives™ on Stack Overflow

php passing arguments in constructor

2 Answers 2

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related