Enable Jacoco for GitLab unit test job

See merge request !145

Use constants for HTTP methods and header names

See merge request !143

Add support to build and run on Java 11

See merge request !142

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

The Nashorn ScriptEngine is deprecated since JDK 11 as described by
JEP 335. We now include GraalJS as a replacement.

Related docs:
* https://github.com/graalvm/graaljs.
* https://openjdk.java.net/jeps/174.
* https://openjdk.java.net/jeps/335.

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

This workaround can be removed when a new plugin version is released.

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

Java 11 removes Java EE modules as described by JEP 320. Since we depend
on some of those modules those have now to be added explicitly.

The following affected APIs are used by us directly or indirectly:
* javax.annotation
* javax.xml.bind

Some workarounds had to be added for now to fix issues caused by
dependencies which include conflicting APIs.

See https://openjdk.java.net/jeps/320.

Add tracing on incoming messages

See merge request !139

Add bean validation for entity classes

See merge request !138

Validation is performed before entities are persisted using Hibernate
Validator, the reference implementation for Bean Validation 2.0
(JSR 380).

Refs #23.

Use YAML instead of properties format for config

See merge request !137

The location of `application.yml` can now be set via environment
variable `ARSNOVA_CONFIG_DIR` or Java system property
`arsnova.config-dir`.

For most of the configuration property classes are now used instead of
using property injection via @Value.

Apply code conventions based on Google's Java conventions and enforce them via Checkstyle

See merge request !136

Checks for missing Javadocs are run separately and do not cause build
failures for now.

* Removed obsolete commented-out code.
* Fixed indentation of commented out code.
* Added FIXME comments where old code still has to be migrated.

Rules:
* (naming) ParameterName
* (naming) LocalVariableName
* (naming) MemberName
* (naming) InterfaceTypeParameterName
* (naming) AbbreviationAsWordInName
* (modifier) ModifierOrder
* (blocks) NeedBraces
* (coding) OneStatementPerLine
* (coding) MultipleVariableDeclarations
* (coding) VariableDeclarationUsageDistance
* (coding) MissingSwitchDefault
* (coding) OverloadMethodsDeclarationOrder
* (extension) RightCurlySame
* (misc) ArrayTypeStyle

The following conventions have been applied:
* Rule: (sizes) LineLength

The following conventions have been applied:
* Fix indentation levels
  Rules: (indentation) Indentation, (regexp) RegexpSinglelineJava
* Always use operators at the beginning of a new line if there is a line
  break.
  Rule: (whitespace) OperatorWrap
* Always use empty lines between method declarations.
  Rule: (whitespace) EmptyLineSeparator
* Always use new line between annotation and field/method declaration.
  Rule: (extension) AnnotationLocationMostCases
* Additional rules:
    * (whitespace) ParenPad
    * (whitespace) GenericWhitespace
    * (blocks) LeftCurly
    * (extension) RightCurlyAlone

The following conventions have been applied:
* Import order: static packages -> other packages -> ARSnova packages
  Rule: (imports) CustomImportOrder
* New line between import package groups
  Rule: (imports) CustomImportOrder
* All packages are imported explicitly (no wildcard imports)
  Rule: (imports) AvoidStarImport
* New line between license comment and package line
  Rule: (whitespace) EmptyLineSeparator

The updated Maven plugin fixes compatibilty with Checkstyle >= 8.2.
Checkstyle >= 8.11 is needed for support of the `LambdaParameterName`
rule used by Google style checks.

Updated Checkstyle to version 8.14 to support all rules.

Add placeholder in reset mail for reset key

See merge request !135

Room moderators

See merge request !131

Daniel Gerhardt authored 5 years ago and Daniel Gerhardt committed 5 years ago

This also fixes the read permission room owners. They now have read
access to closed rooms.

Daniel Gerhardt authored 5 years ago and Daniel Gerhardt committed 5 years ago

The `/room/find` endpoint has been extended to support querying rooms by
moderator user ID. It can be used in combination with the `ownerId`
property.

Daniel Gerhardt authored 5 years ago and Daniel Gerhardt committed 5 years ago

Moderators can assist a room owner in managing the room. This commit
adds API and persistence support for moderators.

Add placeholder in activation mail for activation key

See merge request !133