@Ocramius can you confirm that on this function :

function1()
{
    return 'foo';
}

getBody is :

return 'foo';

getContent is :

function1()
{
    return 'foo';
}

?

@blanchonvincent never used that API - are there no other tests for it?

@Ocramius The prototype is

public function getContents($includeDocBlock = true)

So, if the contents is :

{
   return ....;
}

what is the contents with docblock ? I think there is a bug, and the contents must be :

function foo()
{
   return ....;
}

and with docblock :

/**
 * bla bla bla
 */
function foo()
{
   return ....;
}

@blanchonvincent yep, I agree on that. I'd just first check what the consumers of that method are before applying any change.

@Ocramius, Ok, let me know

@blanchonvincent no, I was suggesting you to grep for its usage :)

@Ocramius, Look never used, and $useDockBlock param doesn't work, and lot of time, the function return an empty result ... so I fix that.

PR is finished -- better code coverage & lot of fixs

@Ocramius since you did original review; could you please review a last time?

@blanchonvincent Please rebase the PR for fix merge conflicts

Merged to develop for release with 2.3.0, as it represents new functionality. I took care of the merge conflicts.