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:codest... 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, because both normal and fluent templates returns $this. So what is the point of having the option to chose between normal and fluent setter template? In the screenshot below, we see that we have an option to choose fluent or normal setter. Greetings, Dragan
