search

Yolgie / jku-lan-se

http://informatik.jku.at/students/lan/
1 stars 0 forks source link

54 refactoring #57

Open woikal opened 8 years ago

woikal commented 8 years ago

Untested! Seperation of Beans, Services into packages, seperate (Web)Controllers Missing: visibility of field may be changed.

Yolgie commented 8 years ago

:+1:

Yolgie commented 8 years ago

Hier werden allerdings in der GUI viel mehr Änderungen angezeigt als notwendig...

woikal commented 8 years ago

wie meinen?

SecMeyo commented 8 years ago

rename converter.java > Converter.java im Output seh ich keinen Stylesheet mehr, soll das so?

TTH-Someone-stole-my-name commented 8 years ago

OK, ich hab mich jetzt mal hingesetzt und diesen Branch mit dem jUnitDebug Branch kombiniert --> Jetzt haben wir da drinnen ein paar Beispiele für funktionierende UnitTests. Allerdings hab ich dabei auch einige neue Changes eingearbeitet --> Ihr werden den Branch also noch ein zweites Mal reviewen müssen. Ich werd heute noch aufm Master rebasen, damit wir den auch automatisch mergen können.

woikal commented 8 years ago

:+1: Hab das refoctory-refit von TTH jetzt nochmal eingehend durchgesehn und würd es so übernehmen.