Skip to content

Extract method (em) not work on static method #37

Open
@alquerci

Description

@alquerci

Hi,

Firstly, thanks for this great Vim plugin.

Be curious regarding issue #31

Note: I personally never used extract method on static method.


Before

    public static function testExtractMethodOnStatic($message)
    {   
        // Make a very cool wave with the message
        for ($i = 0; $i < strlen($message); $i++) {
            $message[$i] = $i % 2 ? strtoupper($message[$i]) : strtolower($message[$i]);
        }
    }

After

    public static function testExtractMethodOnStatic($message)
    {
        // Make a very cool wave with the message
        $message = $this->makeWaveWithMessage($message);
    }

    private function makeWaveWithMessage($message)
    {
        for ($i = 0; $i < strlen($message); $i++) {
            $message[$i] = $i % 2 ? strtoupper($message[$i]) : strtolower($message[$i]);
        }
        return $message;
    }

Expected

    public static function testExtractMethodOnStatic($message)
    {   
        // Make a very cool wave with the message
        $message = self::makeWaveWithMessage($message);
    }

    private static function makeWaveWithMessage($message)
    {   
        for ($i = 0; $i < strlen($message); $i++) {
            $message[$i] = $i % 2 ? strtoupper($message[$i]) : strtolower($message[$i]);
        }
        return $message;
    }

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions