Micke1101 / OSTicket-plugin-Merging

12 stars 5 forks source link

PSR-1 and PSR-2 #2

Open BastianBalthasarBux opened 6 years ago

BastianBalthasarBux commented 6 years ago

Hey, many thanks for that plugin! For me it works on latest osticket release.

The following issues I am going to create are just suggestions on how to enhance the plugin and its code quality.

First of all: what about applying coding-styles of PSR-1 and PSR-2, as far as possible? At least the things about control-structures like if-clauses would be really nice!

Micke1101 commented 6 years ago

Glad you like the plugin, unfortuneatly I've not standarized the code as I've had issues thinking of how to properly enhance the functionality of the plugin due to limitations in osTicket.