Exceptions are now always passed to the logger. The logger decides based
on configuration whether and how they are logged. Logging messages were
adjusted for consistency.

Additionally, remove unused loggers.

* Remove unused code
* Optimize imports (group 10+ and static imports)
* Remove redundant variable assignments
* Replace explicit type parameters with <>
* Remove redundant catch blocks
* Use varargs instead of explicit array instantiation
* Weaken visibility of methods and fields
* Simplify conditions
* Do not use StringBuilder for simple concatenations
* Fix code style
* Remove invalid Javadoc

This simplifies analysis of monitoring data.

fix bug where abstention questions are not exported

implement: via two booleans you can now decide whether AnswerStatistics and FeedbackQuestions shall be exported, too

add constructors to transport classes to parse "main"class

remove swagger annotation from feedbackcontroller, coursecontroller and getter and setter for sortOrder

remove spingfox for ApiIgnore direction
use swagger-maven-plugin to generate swagger.json before compiling rather than
 dynamically generate it on request

Daniel Gerhardt authored 9 years ago and Tom Käsler committed 9 years ago

Reasons:
* not used consistently
* Java signatures are not helpful
* Return type should not be hard-coded in a String

Statistics model now correctly uses value instead of notes for
documentation of properties.

Daniel Gerhardt authored 9 years ago and Tom Käsler committed 9 years ago

* Fix inconsistent whitespaces
* Remove unused imports

Tjark Wilhelm Hoeck authored 9 years ago and Tom Käsler committed 9 years ago

added labeled deprecated rest endpoints to api documentation, commentet and included the feedbackController

All feature settings are replaced by the received object so the use of
PATCH does not make sense here. The session is "patched" but the HTTP
verb always relates to the URL which refers explicitly to features in
this case.

See https://tools.ietf.org/html/rfc5789.