Argument $variable is missing from the Docblock of method

[wesdean]

I am getting an argument missing error for the first argument of every method with arguments. Here is a copy of my doc block and method.

/**
* Sets configuration flags.
*
* @param String $flag The name of the flag to set.
* @param Boolean $value The value of the flag.
* @param Boolean $auto_run Whether to execute Config::run() after setting the flag.
*/
public static function setFlag($flag, $value, $auto_run = false) {
$value = !!$value;

    if ($auto_run && $value == self::varDefault(self::$flags[$flag])) $auto_run = false;

    self::$flags[$flag] = !!$value;
    if ($auto_run) self::run();
}

[georaldc]

This should have been resolved already with the fix for Issue #1460

[wesdean]

I understand that this issue should have been fixed, but I am still having trouble with it on version 2.8.1.
I am also running PHP 5.4 on CentOS 5. I get no error from the command output, just on the compilation errors report in the generated documentation.

[georaldc]

Hmm, doesn't look like the fix made it to the latest tagged release. I guess you could wait for the next release, or just clone the latest develop branch

[wesdean]

Okay, thanks. Are there any instructions on installing phpDocumentor from source?

[georaldc]

If you're using composer, I think the following should point to the latest develop branch:

"phpdocumentor/phpdocumentor": "2.9.*@dev"

[mvriel]

The contribution was provided against develop and not master; since we are making preparations for going 3.0 with phpDocumentor on the develop branch it will be some time before that will be released.

What needs to happen is that the commits of PR #1469 need to be cherrypicked or re-implemented against master.

Tomorrow is a hackathon where we will hopefully have several new contributions; I am going to flag this issue as an easy pick so they can help me with this and all bugs that are solved during the hackathon can be merged after it and then a new point release can be made.

[georaldc]

Sounds fun.

I just realized that PR #1469 had my other, non personal github account attached to it. Forgot to change the email address in my work computer, no wonder I could see notifications in my other account T_T

[b4oshany]

I'm having the same issue with

    /**
    * Abort all php process and return HTTP status code 404 to the connected client.
    * @param string $message - HTTP Response message to output to the client.
        It is default to Not Found.
    * @param int $status_code - HTTP Response status code. The status code is
        default to 404, which means Not Found.
    * @param string $status_text - HTTP Response status text. The status text is
        default to Not Found.
    */
    public static function abort($message = "Not Found", $status_code=404, $status_text="Not Found"){
        if(!http\Request::is_async()){
            echo self::$twig->render('404.html',
                                     array('message'=>$message)
                                   );
        }
        http\Response::abort($message, $status_code, $status_text);
    }

[sascha-egerer]

Anyone to pick this?

[sascha-egerer]

There is currently no branch for version 2.8.x to do a pull request against it, right?

[mvriel]

All changes that need to be deployed as part of 2.* can be PRed to Master; develop is used for the work on version 3