Hi everybody.

I was looking at the wiki page about the code templates

https://wiki.app.infra.gs.xtrav.de/doku.php?id=root:traso:entwicklung:codestyle:general

I have noticed that the code for Code PHP Setter Method has a little change in comparison to the original template in the PHPstorm

/**

* @param ${TYPE_HINT} $${PARAM_NAME}

#if (${STATIC} != "static")

*
          @return self

#end

*/

public ${STATIC} function set${NAME}($${PARAM_NAME})

{

#if

(${STATIC}
        ==

"static")

self::$${FIELD_NAME} = $${PARAM_NAME};

#else

$this

->${FIELD_NAME}
        = $${PARAM_NAME};

return $this;

#end

}

If you are returning $this, it is called FLUENT setter....so that you can chain setters on the object.

This is already a part of the previous template in the wiki page

Code PHP Fluent Setter Method

/**
 * @param ${TYPE_HINT} $${PARAM_NAME}
 * @return self
 */
public function
            set${NAME}($${PARAM_NAME})
{
    $this->${FIELD_NAME}
              = $${PARAM_NAME};
    return $this;
}

            So In my opinion, no need to return $this from a
            method which is not made with a FLUENT template.

            In the screenshot below, we see that we have an option to
            choose fluent or normal setter.

            Greetings,

            Dragan