Scapegoat Inspections
Errors
0
Warnings
179
Infos
103
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.AdminController.scala:48
Warning
Unused constructor parameter
com.sksamuel.scapegoat.inspections.unneccesary.UnusedMethodParameter
Unused constructor parameter (userService)
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.AdminController.scala:49
Warning
Unused constructor parameter
com.sksamuel.scapegoat.inspections.unneccesary.UnusedMethodParameter
Unused constructor parameter (emailService)
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.AdminController.scala:51
Warning
Unused constructor parameter
com.sksamuel.scapegoat.inspections.unneccesary.UnusedMethodParameter
Unused constructor parameter (collectionsService)
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.CollectionsController.scala:39
Warning
Unused constructor parameter
com.sksamuel.scapegoat.inspections.unneccesary.UnusedMethodParameter
Unused constructor parameter (emailService)
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.ConfiguredController.scala:28
Warning
Unused constructor parameter
com.sksamuel.scapegoat.inspections.unneccesary.UnusedMethodParameter
Unused constructor parameter (portalConfig)
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.CswController.scala:66
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
CswController.this.app.resource("csw/transaction.insert.xml").get
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.CswController.scala:67
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
CswController.this.app.resource("csw/transaction.update.xml").get
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.CswController.scala:68
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
CswController.this.app.resource("csw/transaction.delete.xml").get
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.CswController.scala:98
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
validValuesOption.get
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.CswController.scala:127
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
node.attribute("name").get
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.CswController.scala:128
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
node.attribute("name").get
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.CswController.scala:127
Warning
Unnecessary toString
com.sksamuel.scapegoat.inspections.unneccesary.UnnecessaryToString
Unnecessary toString on instanceo of String: xml.this.NodeSeq.seqToNodeSeq(node.attribute("name").get).text.toString
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.CswController.scala:250
Warning
Unnecessary toString
com.sksamuel.scapegoat.inspections.unneccesary.UnnecessaryToString
Unnecessary toString on instanceo of String: userMetaEntry.originaluuid.toString
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.CswController.scala:333
Warning
Unnecessary toString
com.sksamuel.scapegoat.inspections.unneccesary.UnnecessaryToString
Unnecessary toString on instanceo of String: userMetaEntry.originaluuid.toString
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.HomeController.scala:53
Warning
Unused method parameter
com.sksamuel.scapegoat.inspections.unneccesary.UnusedMethodParameter
Unused method parameter (val all: String = _)
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.HomeController.scala:80
Warning
Unused method parameter
com.sksamuel.scapegoat.inspections.unneccesary.UnusedMethodParameter
Unused method parameter (val fields: Option[String] = _)
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.SosDataController.scala:361
Warning
Catch exception
com.sksamuel.scapegoat.inspections.exception.CatchException
Did you intend to catch all exceptions, consider catching a more specific exception class: try {
sheets.fold[Option[models.ErrorResult]]({
SosDataController.this.logger.error("Couldn\'t extract OM2 into SpreadSheet from this SOS GetObservation response");
scala.Some.apply[models.ErrorResult](models.ErrorResult.apply("Couldn\'t extract OM2 into SpreadSheet from this SOS GetObserv
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.SosDataController.scala:335
Warning
Null parameter
com.sksamuel.scapegoat.inspections.nulls.NullParameter
Null is used as a method parameter: scala.xml.XML.save(tmpFile.getAbsolutePath(), wml2.get.head, "UTF-8", true, null)
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.SosDataController.scala:119
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
response.header("Content-Type").get
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.SosDataController.scala:120
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
response.header("Content-Type").get
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.SosDataController.scala:122
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
response.header("Content-Type").get
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.SosDataController.scala:126
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
timeseries.responseFormat.get
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.SosDataController.scala:250
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
response.header("Content-Type").get
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.SosDataController.scala:251
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
response.header("Content-Type").get
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.SosDataController.scala:253
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
response.header("Content-Type").get
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.SosDataController.scala:335
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
wml2.get
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.SosDataController.scala:484
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
response.header("Content-Type").get
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.SosDataController.scala:485
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
response.header("Content-Type").get
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.SosDataController.scala:486
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
response.header("Content-Type").get
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.SosDataController.scala:490
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
models.sosdata.SosCapabilities.fromXml(response.xml, sosUrl).get
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.SosDataController.scala:452
Info
Match instead of partial function
com.sksamuel.scapegoat.inspections.matching.PartialFunctionInsteadOfMatch
A map match can be replaced with a partial function for greater readability: responseFuture.map[play.api.mvc.Result](((response: Either[models.ErrorResult,models.sosdata.SosCapabilities]) => response match {
case (b: models.sosdata.SosCapabilities)scala.util.Right[models.ErrorResult,models.sosdata.SosCapabilities]((sosCapabilities @ _)) => SosDataController.this.Ok.apply[play.api.libs.json.JsValue](sosCapabilities.toJson())(http.this.Writeable.writeableOf_JsValue(mvc.this.Codec.utf_8)).as(SosDataController.this.JSON)
case (a: models.ErrorResult)scala.util.Left[models
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.SosDataController.scala:156
Warning
Use of Traversable.head
com.sksamuel.scapegoat.inspections.collections.TraversableHead
tvp.head
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.SosDataController.scala:335
Warning
Use of Traversable.head
com.sksamuel.scapegoat.inspections.collections.TraversableHead
wml2.get.head
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.UserController.scala:504
Warning
Use of asInstanceOf
com.sksamuel.scapegoat.inspections.unsafe.AsInstanceOf
asInstanceOf used near gAuthPayload.get("name").asInstanceOf[String]. Consider using pattern matching.
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.UserController.scala:505
Warning
Use of asInstanceOf
com.sksamuel.scapegoat.inspections.unsafe.AsInstanceOf
asInstanceOf used near gAuthPayload.get("family_name").asInstanceOf[String]. Consider using pattern matching.
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.UserController.scala:506
Warning
Use of asInstanceOf
com.sksamuel.scapegoat.inspections.unsafe.AsInstanceOf
asInstanceOf used near gAuthPayload.get("given_name").asInstanceOf[String]. Consider using pattern matching.
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.UserController.scala:45
Info
Redundant final modifier on case class
com.sksamuel.scapegoat.inspections.RedundantFinalModifierOnCaseClass
Case classes cannot be extended, final modifer is redundant
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.UserController.scala:61
Info
Redundant final modifier on case class
com.sksamuel.scapegoat.inspections.RedundantFinalModifierOnCaseClass
Case classes cannot be extended, final modifer is redundant
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.UserGroupController.scala:37
Warning
Unused constructor parameter
com.sksamuel.scapegoat.inspections.unneccesary.UnusedMethodParameter
Unused constructor parameter (emailService)
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.UserGroupController.scala:39
Warning
Unused constructor parameter
com.sksamuel.scapegoat.inspections.unneccesary.UnusedMethodParameter
Unused constructor parameter (googleService)
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.csw.CswTransactions.scala:74
Info
Redundant final modifier on case class
com.sksamuel.scapegoat.inspections.RedundantFinalModifierOnCaseClass
Case classes cannot be extended, final modifer is redundant
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.csw.CswTransactions.scala:103
Info
Redundant final modifier on case class
com.sksamuel.scapegoat.inspections.RedundantFinalModifierOnCaseClass
Case classes cannot be extended, final modifer is redundant
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.csw.CswTransactions.scala:125
Info
Redundant final modifier on case class
com.sksamuel.scapegoat.inspections.RedundantFinalModifierOnCaseClass
Case classes cannot be extended, final modifer is redundant
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.package.scala:80
Info
Duplicated Import
com.sksamuel.scapegoat.inspections.imports.DuplicateImport
play.api.libs.functional.syntax._
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.package.scala:82
Info
Duplicated Import
com.sksamuel.scapegoat.inspections.imports.DuplicateImport
play.api.libs.functional.syntax._
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.package.scala:83
Info
Duplicated Import
com.sksamuel.scapegoat.inspections.imports.DuplicateImport
play.api.libs.functional.syntax._
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.package.scala:85
Info
Duplicated Import
com.sksamuel.scapegoat.inspections.imports.DuplicateImport
play.api.libs.functional.syntax._
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.package.scala:86
Info
Duplicated Import
com.sksamuel.scapegoat.inspections.imports.DuplicateImport
play.api.libs.functional.syntax._
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.package.scala:88
Info
Duplicated Import
com.sksamuel.scapegoat.inspections.imports.DuplicateImport
play.api.libs.functional.syntax._
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.package.scala:89
Info
Duplicated Import
com.sksamuel.scapegoat.inspections.imports.DuplicateImport
play.api.libs.functional.syntax._
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.package.scala:90
Info
Duplicated Import
com.sksamuel.scapegoat.inspections.imports.DuplicateImport
play.api.libs.functional.syntax._
.home.travis.build.ZGIS.smart-portal-backend.app.controllers.package.scala:92
Info
Duplicated Import
com.sksamuel.scapegoat.inspections.imports.DuplicateImport
play.api.libs.functional.syntax._
.home.travis.build.ZGIS.smart-portal-backend.app.models.ErrorResult.scala:29
Info
Redundant final modifier on case class
com.sksamuel.scapegoat.inspections.RedundantFinalModifierOnCaseClass
Case classes cannot be extended, final modifer is redundant
.home.travis.build.ZGIS.smart-portal-backend.app.models.gmd.MdMetadata.scala:264
Warning
Use of asInstanceOf
com.sksamuel.scapegoat.inspections.unsafe.AsInstanceOf
asInstanceOf used near finalXml.asInstanceOf[scala.xml.Node]. Consider using pattern matching.
.home.travis.build.ZGIS.smart-portal-backend.app.models.gmd.MdMetadata.scala:111
Info
Redundant final modifier on case class
com.sksamuel.scapegoat.inspections.RedundantFinalModifierOnCaseClass
Case classes cannot be extended, final modifer is redundant
.home.travis.build.ZGIS.smart-portal-backend.app.models.gmd.MdMetadata.scala:324
Warning
Unused method parameter
com.sksamuel.scapegoat.inspections.unneccesary.UnusedMethodParameter
Unused method parameter (val node: scala.xml.Node = _)
.home.travis.build.ZGIS.smart-portal-backend.app.models.gmd.MdMetadataCitation.scala:60
Warning
Null assignment
com.sksamuel.scapegoat.inspections.nulls.NullAssignment
Null assignment on line 60
.home.travis.build.ZGIS.smart-portal-backend.app.models.gmd.MdMetadataCitation.scala:44
Info
Redundant final modifier on case class
com.sksamuel.scapegoat.inspections.RedundantFinalModifierOnCaseClass
Case classes cannot be extended, final modifer is redundant
.home.travis.build.ZGIS.smart-portal-backend.app.models.gmd.MdMetadataDistribution.scala:87
Warning
Null assignment
com.sksamuel.scapegoat.inspections.nulls.NullAssignment
Null assignment on line 87
.home.travis.build.ZGIS.smart-portal-backend.app.models.gmd.MdMetadataDistribution.scala:40
Info
Redundant final modifier on case class
com.sksamuel.scapegoat.inspections.RedundantFinalModifierOnCaseClass
Case classes cannot be extended, final modifer is redundant
.home.travis.build.ZGIS.smart-portal-backend.app.models.gmd.MdMetadataExtent.scala:68
Warning
Null assignment
com.sksamuel.scapegoat.inspections.nulls.NullAssignment
Null assignment on line 68
.home.travis.build.ZGIS.smart-portal-backend.app.models.gmd.MdMetadataExtent.scala:40
Info
Redundant final modifier on case class
com.sksamuel.scapegoat.inspections.RedundantFinalModifierOnCaseClass
Case classes cannot be extended, final modifer is redundant
.home.travis.build.ZGIS.smart-portal-backend.app.models.gmd.MdMetadataResponsibleParty.scala:97
Warning
Null assignment
com.sksamuel.scapegoat.inspections.nulls.NullAssignment
Null assignment on line 97
.home.travis.build.ZGIS.smart-portal-backend.app.models.gmd.MdMetadataResponsibleParty.scala:40
Info
Redundant final modifier on case class
com.sksamuel.scapegoat.inspections.RedundantFinalModifierOnCaseClass
Case classes cannot be extended, final modifer is redundant
.home.travis.build.ZGIS.smart-portal-backend.app.models.metadata.ValidValues.scala:59
Warning
Java conversions
com.sksamuel.scapegoat.inspections.collections.JavaConversionsUse
Use of java conversions can lead to unusual behaviour. It is recommended to use JavaConverters
.home.travis.build.ZGIS.smart-portal-backend.app.models.metadata.ValidValues.scala:48
Info
List.size is O(n)
com.sksamuel.scapegoat.inspections.collections.ListSize
List.size is O(n). Consider using a different data type with O(1) size lookup such as Vector or Array.
.home.travis.build.ZGIS.smart-portal-backend.app.models.metadata.ValidValues.scala:48
Info
List.size is O(n)
com.sksamuel.scapegoat.inspections.collections.ListSize
List.size is O(n). Consider using a different data type with O(1) size lookup such as Vector or Array.
.home.travis.build.ZGIS.smart-portal-backend.app.models.metadata.ValidValues.scala:49
Info
List.size is O(n)
com.sksamuel.scapegoat.inspections.collections.ListSize
List.size is O(n). Consider using a different data type with O(1) size lookup such as Vector or Array.
.home.travis.build.ZGIS.smart-portal-backend.app.models.metadata.ValidValues.scala:49
Info
List.size is O(n)
com.sksamuel.scapegoat.inspections.collections.ListSize
List.size is O(n). Consider using a different data type with O(1) size lookup such as Vector or Array.
.home.travis.build.ZGIS.smart-portal-backend.app.models.metadata.ValidValues.scala:45
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
ValidValues.this.descriptions.get
.home.travis.build.ZGIS.smart-portal-backend.app.models.metadata.ValidValues.scala:49
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
ValidValues.this.descriptions.get
.home.travis.build.ZGIS.smart-portal-backend.app.models.metadata.ValidValues.scala:49
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
ValidValues.this.descriptions.get
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcContextDAO.scala:459
Info
List.size is O(n)
com.sksamuel.scapegoat.inspections.collections.ListSize
List.size is O(n). Consider using a different data type with O(1) size lookup such as Vector or Array.
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcContextDAO.scala:465
Info
List.size is O(n)
com.sksamuel.scapegoat.inspections.collections.ListSize
List.size is O(n). Consider using a different data type with O(1) size lookup such as Vector or Array.
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcContextDAO.scala:790
Info
List.size is O(n)
com.sksamuel.scapegoat.inspections.collections.ListSize
List.size is O(n). Consider using a different data type with O(1) size lookup such as Vector or Array.
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcContextDAO.scala:798
Info
List.size is O(n)
com.sksamuel.scapegoat.inspections.collections.ListSize
List.size is O(n). Consider using a different data type with O(1) size lookup such as Vector or Array.
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcContextDAO.scala:805
Info
List.size is O(n)
com.sksamuel.scapegoat.inspections.collections.ListSize
List.size is O(n). Consider using a different data type with O(1) size lookup such as Vector or Array.
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcContextDAO.scala:854
Info
List.size is O(n)
com.sksamuel.scapegoat.inspections.collections.ListSize
List.size is O(n). Consider using a different data type with O(1) size lookup such as Vector or Array.
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcContextDAO.scala:860
Info
List.size is O(n)
com.sksamuel.scapegoat.inspections.collections.ListSize
List.size is O(n). Consider using a different data type with O(1) size lookup such as Vector or Array.
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcContextDAO.scala:338
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
owcContext.creatorApplication.get
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcContextDAO.scala:341
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
owcContext.creatorApplication.get
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcContextDAO.scala:344
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
owcContext.creatorApplication.get
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcContextDAO.scala:361
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
owcContext.creatorDisplay.get
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcContextDAO.scala:364
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
owcContext.creatorDisplay.get
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcContextDAO.scala:367
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
owcContext.creatorDisplay.get
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcContextDAO.scala:100
Warning
Unnecessary toString
com.sksamuel.scapegoat.inspections.unneccesary.UnnecessaryToString
Unnecessary toString on instanceo of String: id.toString
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcContextDAO.scala:518
Warning
Unused method parameter
com.sksamuel.scapegoat.inspections.unneccesary.UnusedMethodParameter
Unused method parameter (val userAccountSub: String = _)
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcContextDAO.scala:698
Warning
Unused method parameter
com.sksamuel.scapegoat.inspections.unneccesary.UnusedMethodParameter
Unused method parameter (val user: models.users.User = _)
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcOfferingDAO.scala:419
Warning
Use of asInstanceOf
com.sksamuel.scapegoat.inspections.unsafe.AsInstanceOf
asInstanceOf used near OwcAuthorDAO.findOwcAuthorByUuid(uid)(connection).asInstanceOf[Option[A]]. Consider using pattern matching.
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcOfferingDAO.scala:420
Warning
Use of asInstanceOf
com.sksamuel.scapegoat.inspections.unsafe.AsInstanceOf
asInstanceOf used near OwcCategoryDAO.findOwcCategoryByUuid(uid)(connection).asInstanceOf[Option[A]]. Consider using pattern matching.
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcOfferingDAO.scala:421
Warning
Use of asInstanceOf
com.sksamuel.scapegoat.inspections.unsafe.AsInstanceOf
asInstanceOf used near OwcLinkDAO.findOwcLinkByUuid(uid)(connection).asInstanceOf[Option[A]]. Consider using pattern matching.
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcOfferingDAO.scala:422
Warning
Use of asInstanceOf
com.sksamuel.scapegoat.inspections.unsafe.AsInstanceOf
asInstanceOf used near OwcContentDAO.findOwcContentByUuid(uid)(connection).asInstanceOf[Option[A]]. Consider using pattern matching.
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcOfferingDAO.scala:423
Warning
Use of asInstanceOf
com.sksamuel.scapegoat.inspections.unsafe.AsInstanceOf
asInstanceOf used near OwcCreatorApplicationDAO.findOwcCreatorApplicationByUuid(uid)(connection).asInstanceOf[Option[A]]. Consider using pattern matching.
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcOfferingDAO.scala:424
Warning
Use of asInstanceOf
com.sksamuel.scapegoat.inspections.unsafe.AsInstanceOf
asInstanceOf used near OwcCreatorDisplayDAO.findOwcCreatorDisplayByUuid(uid)(connection).asInstanceOf[Option[A]]. Consider using pattern matching.
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcOfferingDAO.scala:425
Warning
Use of asInstanceOf
com.sksamuel.scapegoat.inspections.unsafe.AsInstanceOf
asInstanceOf used near OwcStyleSetDAO.findOwcStyleSetByUuid(uid)(connection).asInstanceOf[Option[A]]. Consider using pattern matching.
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcOfferingDAO.scala:426
Warning
Use of asInstanceOf
com.sksamuel.scapegoat.inspections.unsafe.AsInstanceOf
asInstanceOf used near OwcOperationDAO.findOwcOperationByUuid(uid)(connection).asInstanceOf[Option[A]]. Consider using pattern matching.
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcOfferingDAO.scala:427
Warning
Use of asInstanceOf
com.sksamuel.scapegoat.inspections.unsafe.AsInstanceOf
asInstanceOf used near OwcOfferingDAO.this.findOwcOfferingByUuid(uid)(connection).asInstanceOf[Option[A]]. Consider using pattern matching.
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcOfferingDAO.scala:416
Info
filter(_.isDefined).map(_.get)
com.sksamuel.scapegoat.inspections.collections.FilterOptionAndGet
.filter(_.isDefined).map(_.get) can be replaced with flatten: potUuids.map[Option[java.util.UUID], Seq[Option[java.util.UUID]]](((x$44: String) => utils.StringUtils.OptionUuidConverters(x$44).toUuidOption))(collection.this.Seq.canBuildFrom[Option[java.util.UUID]]).filter(((x$45: Option[java.util.UUID]) => x$45.isDefined)).map[java.util.UUID, Seq[java.util.UUID]](((x$46: Option[java.util.UUID]) => x$46.get))
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcOfferingDAO.scala:431
Info
filter(_.isDefined).map(_.get)
com.sksamuel.scapegoat.inspections.collections.FilterOptionAndGet
.filter(_.isDefined).map(_.get) can be replaced with flatten: uuids.map[Option[A], Seq[Option[A]]](((uid: java.util.UUID) => a match {
case (a @ (_: info.smart.models.owc100.OwcAuthor)) => OwcAuthorDAO.findOwcAuthorByUuid(uid)(connection).asInstanceOf[Option[A]]
case (a @ (_: info.smart.models.owc100.OwcCategory)) => OwcCategoryDAO.findOwcCategoryByUuid(uid)(connection).asInstanceOf[Option[A]]
case (a @ (_: info.smart.models.owc100.OwcLink)) => OwcLinkDAO.findOwcLinkByUuid(uid)(connection).asInstanceOf[Option[A]]
case (a @ (_: info.smart.models.owc
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcOfferingDAO.scala:365
Info
List.size is O(n)
com.sksamuel.scapegoat.inspections.collections.ListSize
List.size is O(n). Consider using a different data type with O(1) size lookup such as Vector or Array.
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcOfferingDAO.scala:372
Info
List.size is O(n)
com.sksamuel.scapegoat.inspections.collections.ListSize
List.size is O(n). Consider using a different data type with O(1) size lookup such as Vector or Array.
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcOfferingDAO.scala:379
Info
List.size is O(n)
com.sksamuel.scapegoat.inspections.collections.ListSize
List.size is O(n). Consider using a different data type with O(1) size lookup such as Vector or Array.
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcOfferingDAO.scala:416
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
x$46.get
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcOfferingDAO.scala:431
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
x$48.get
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcOfferingDAO.scala:434
Info
Prefer Seq.empty
com.sksamuel.scapegoat.inspections.collections.PreferSeqEmpty
Seq[T]() creates a new instance. Consider Seq.empty which does not allocate a new object. collection.this.Seq.apply[Nothing]()
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcOperationDAO.scala:99
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
owcOperation.request.get
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcOperationDAO.scala:101
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
owcOperation.request.get
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcOperationDAO.scala:104
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
owcOperation.request.get
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcOperationDAO.scala:111
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
owcOperation.result.get
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcOperationDAO.scala:113
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
owcOperation.result.get
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcOperationDAO.scala:116
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
owcOperation.result.get
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcResourceDAO.scala:637
Info
List.size is O(n)
com.sksamuel.scapegoat.inspections.collections.ListSize
List.size is O(n). Consider using a different data type with O(1) size lookup such as Vector or Array.
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcResourceDAO.scala:647
Info
List.size is O(n)
com.sksamuel.scapegoat.inspections.collections.ListSize
List.size is O(n). Consider using a different data type with O(1) size lookup such as Vector or Array.
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcResourceDAO.scala:654
Info
List.size is O(n)
com.sksamuel.scapegoat.inspections.collections.ListSize
List.size is O(n). Consider using a different data type with O(1) size lookup such as Vector or Array.
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcResourceDAO.scala:661
Info
List.size is O(n)
com.sksamuel.scapegoat.inspections.collections.ListSize
List.size is O(n). Consider using a different data type with O(1) size lookup such as Vector or Array.
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcStyleSetDAO.scala:91
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
owcStyleSet.content.get
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcStyleSetDAO.scala:93
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
owcStyleSet.content.get
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcStyleSetDAO.scala:96
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
owcStyleSet.content.get
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcStyleSetDAO.scala:139
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
owcStyleSet.content.get
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcStyleSetDAO.scala:141
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
owcStyleSet.content.get
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcStyleSetDAO.scala:144
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
owcStyleSet.content.get
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcStyleSetDAO.scala:150
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
toBeDeleted.get
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcStyleSetDAO.scala:194
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
owcStyleSet.content.get
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.OwcStyleSetDAO.scala:196
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
owcStyleSet.content.get
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.package.scala:71
Warning
Use of asInstanceOf
com.sksamuel.scapegoat.inspections.unsafe.AsInstanceOf
asInstanceOf used near value.asInstanceOf[AnyRef]. Consider using pattern matching.
.home.travis.build.ZGIS.smart-portal-backend.app.models.owc.package.scala:97
Warning
Use of asInstanceOf
com.sksamuel.scapegoat.inspections.unsafe.AsInstanceOf
asInstanceOf used near value.asInstanceOf[AnyRef]. Consider using pattern matching.
.home.travis.build.ZGIS.smart-portal-backend.app.models.rdf.CategoryHolder.scala:44
Info
Redundant final modifier on case class
com.sksamuel.scapegoat.inspections.RedundantFinalModifierOnCaseClass
Case classes cannot be extended, final modifer is redundant
.home.travis.build.ZGIS.smart-portal-backend.app.models.rdf.ResearchPGHolder.scala:39
Info
Redundant final modifier on case class
com.sksamuel.scapegoat.inspections.RedundantFinalModifierOnCaseClass
Case classes cannot be extended, final modifer is redundant
.home.travis.build.ZGIS.smart-portal-backend.app.models.rdf.XlsToSparqlRdfConverter.scala:198
Warning
Catch exception
com.sksamuel.scapegoat.inspections.exception.CatchException
Did you intend to catch all exceptions, consider catching a more specific exception class: try {
XlsToSparqlRdfConverter.this.logger.trace(scala.StringContext.apply("before cell ", " 4, after lastParentValue ", "").s(i, lastParentValue));
val description: String = XlsToSparqlRdfConverter.this.getCellValueAsStringOption(i, 4, domainSheet).getOrElse[String]("");
XlsToSparqlRdfConverte
.home.travis.build.ZGIS.smart-portal-backend.app.models.rdf.XlsToSparqlRdfConverter.scala:264
Warning
Catch exception
com.sksamuel.scapegoat.inspections.exception.CatchException
Did you intend to catch all exceptions, consider catching a more specific exception class: try {
val titleName: String = XlsToSparqlRdfConverter.this.getCellValueAsStringOption(i, 0, researchpgSheet).getOrElse[String]("");
val abbrev: String = XlsToSparqlRdfConverter.this.getCellValueAsStringOption(i, 1, researchpgSheet).getOrElse[String]("");
val description: String = XlsToSparqlRd
.home.travis.build.ZGIS.smart-portal-backend.app.models.rdf.XlsToSparqlRdfConverter.scala:372
Warning
Catch exception
com.sksamuel.scapegoat.inspections.exception.CatchException
Did you intend to catch all exceptions, consider catching a more specific exception class: try {
val collectionIdentifier: String = XlsToSparqlRdfConverter.this.getCellValueAsStringOption(1, 1, collectionInfoSheet).getOrElse[String]("");
val hierarchy: String = XlsToSparqlRdfConverter.this.getCellValueAsStringOption(2, 1, collectionInfoSheet).getOrElse[String]("");
val hierarchyPlur
.home.travis.build.ZGIS.smart-portal-backend.app.models.rdf.XlsToSparqlRdfConverter.scala:66
Warning
Null parameter
com.sksamuel.scapegoat.inspections.nulls.NullParameter
Null is used as a method parameter: theRow.!=(null)
.home.travis.build.ZGIS.smart-portal-backend.app.models.rdf.XlsToSparqlRdfConverter.scala:81
Warning
Null parameter
com.sksamuel.scapegoat.inspections.nulls.NullParameter
Null is used as a method parameter: theRow.getCell(cell, RETURN_NULL_AND_BLANK).!=(null)
.home.travis.build.ZGIS.smart-portal-backend.app.models.rdf.XlsToSparqlRdfConverter.scala:147
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
XlsToSparqlRdfConverter.this.getCellValueAsStringOption(i, 0, synonymSheet).get
.home.travis.build.ZGIS.smart-portal-backend.app.models.rdf.XlsToSparqlRdfConverter.scala:148
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
XlsToSparqlRdfConverter.this.getCellValueAsStringOption(i, 1, synonymSheet).get
.home.travis.build.ZGIS.smart-portal-backend.app.models.rdf.XlsToSparqlRdfConverter.scala:182
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
XlsToSparqlRdfConverter.this.getCellValueAsStringOption(i, 0, domainSheet).get
.home.travis.build.ZGIS.smart-portal-backend.app.models.sosdata.SosCapabilities.scala:86
Warning
Catch exception
com.sksamuel.scapegoat.inspections.exception.CatchException
Did you intend to catch all exceptions, consider catching a more specific exception class: try {
nodeSeq.head.label match {
case "Capabilities" => scala.Some.apply[models.sosdata.SosCapabilities](SosCapabilities.apply(nodeSeq.\("ServiceIdentification").\("Title").text, sosUrl, SosCapabilities.this.parseValuesFromOperationsMetadata(nodeSeq.\\("OperationsMetadata").\("Operation").filt
.home.travis.build.ZGIS.smart-portal-backend.app.models.sosdata.SosCapabilities.scala:31
Info
Redundant final modifier on case class
com.sksamuel.scapegoat.inspections.RedundantFinalModifierOnCaseClass
Case classes cannot be extended, final modifer is redundant
.home.travis.build.ZGIS.smart-portal-backend.app.models.sosdata.SosCapabilities.scala:87
Warning
Use of Traversable.head
com.sksamuel.scapegoat.inspections.collections.TraversableHead
nodeSeq.head
.home.travis.build.ZGIS.smart-portal-backend.app.models.sosdata.SosCapabilities.scala:112
Warning
Use of Traversable.head
com.sksamuel.scapegoat.inspections.collections.TraversableHead
nodeSeq.head
.home.travis.build.ZGIS.smart-portal-backend.app.models.sosdata.SosServiceMetadata.scala:83
Warning
Catch exception
com.sksamuel.scapegoat.inspections.exception.CatchException
Did you intend to catch all exceptions, consider catching a more specific exception class: try {
nodeSeq.head.label match {
case "Capabilities" => scala.Some.apply[models.sosdata.SosServiceMetadata](SosServiceMetadata.apply(scala.Some.apply[String](nodeSeq.\("ServiceIdentification").\("Abstract").text), scala.Some.apply[String](nodeSeq.\("ServiceIdentification").\("Fees").text), sca
.home.travis.build.ZGIS.smart-portal-backend.app.models.sosdata.SosServiceMetadata.scala:31
Info
Redundant final modifier on case class
com.sksamuel.scapegoat.inspections.RedundantFinalModifierOnCaseClass
Case classes cannot be extended, final modifer is redundant
.home.travis.build.ZGIS.smart-portal-backend.app.models.sosdata.SosServiceMetadata.scala:84
Warning
Use of Traversable.head
com.sksamuel.scapegoat.inspections.collections.TraversableHead
nodeSeq.head
.home.travis.build.ZGIS.smart-portal-backend.app.models.sosdata.SosServiceMetadata.scala:96
Warning
Use of Traversable.head
com.sksamuel.scapegoat.inspections.collections.TraversableHead
nodeSeq.head
.home.travis.build.ZGIS.smart-portal-backend.app.models.sosdata.SpreadsheetExport.scala:57
Warning
Catch exception
com.sksamuel.scapegoat.inspections.exception.CatchException
Did you intend to catch all exceptions, consider catching a more specific exception class: try {
val updatedTime: java.time.OffsetDateTime = java.time.OffsetDateTime.now(java.time.ZoneId.of(SpreadsheetExport.this.timeZone));
val serviceMeta: String = sosCapabilities.serviceMetadata.map[String](((m: models.sosdata.SosServiceMetadata) => scala.this.Predef.augmentString(scala.StringConte
.home.travis.build.ZGIS.smart-portal-backend.app.models.sosdata.Timeseries.scala:30
Info
Redundant final modifier on case class
com.sksamuel.scapegoat.inspections.RedundantFinalModifierOnCaseClass
Case classes cannot be extended, final modifer is redundant
.home.travis.build.ZGIS.smart-portal-backend.app.models.sosdata.Timeseries.scala:59
Info
Redundant final modifier on case class
com.sksamuel.scapegoat.inspections.RedundantFinalModifierOnCaseClass
Case classes cannot be extended, final modifer is redundant
.home.travis.build.ZGIS.smart-portal-backend.app.models.sosdata.Wml2Export.scala:35
Warning
Catch exception
com.sksamuel.scapegoat.inspections.exception.CatchException
Did you intend to catch all exceptions, consider catching a more specific exception class: try {
val nodeSeq: scala.xml.Elem = scala.xml.XML.loadString(xmlText);
nodeSeq.head.label match {
case "GetObservationResponse" => {
val omMembers: scala.xml.NodeSeq = nodeSeq.\\("OM_Observation").map[scala.xml.Elem, scala.xml.NodeSeq](((node: scala.xml.Node) => {
{
v
.home.travis.build.ZGIS.smart-portal-backend.app.models.sosdata.Wml2Export.scala:79
Info
filter(_.isDefined).map(_.get)
com.sksamuel.scapegoat.inspections.collections.FilterOptionAndGet
.filter(_.isDefined).map(_.get) can be replaced with flatten: omMembers.\\("beginPosition").map[String, scala.collection.immutable.Seq[String]](((node: scala.xml.Node) => node.text))(immutable.this.Seq.canBuildFrom[String]).map[Option[java.time.OffsetDateTime], scala.collection.immutable.Seq[Option[java.time.OffsetDateTime]]](((tpos: String) => utils.GeoDateParserUtils.parseDateStringAsOffsetDateTimeSingle(tpos).toOption))(immutable.this.Seq.canBuildFrom[Option[java.time.OffsetDateTime]]).filter(((x$1: Option[java.time.OffsetDateTime]) => x$1.isDefined)).m
.home.travis.build.ZGIS.smart-portal-backend.app.models.sosdata.Wml2Export.scala:81
Info
filter(_.isDefined).map(_.get)
com.sksamuel.scapegoat.inspections.collections.FilterOptionAndGet
.filter(_.isDefined).map(_.get) can be replaced with flatten: omMembers.\\("endPosition").map[String, scala.collection.immutable.Seq[String]](((node: scala.xml.Node) => node.text))(immutable.this.Seq.canBuildFrom[String]).map[Option[java.time.OffsetDateTime], scala.collection.immutable.Seq[Option[java.time.OffsetDateTime]]](((tpos: String) => utils.GeoDateParserUtils.parseDateStringAsOffsetDateTimeSingle(tpos).toOption))(immutable.this.Seq.canBuildFrom[Option[java.time.OffsetDateTime]]).filter(((x$3: Option[java.time.OffsetDateTime]) => x$3.isDefined)).map
.home.travis.build.ZGIS.smart-portal-backend.app.models.sosdata.Wml2Export.scala:79
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
x$2.get
.home.travis.build.ZGIS.smart-portal-backend.app.models.sosdata.Wml2Export.scala:81
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
x$4.get
.home.travis.build.ZGIS.smart-portal-backend.app.models.sosdata.Wml2Export.scala:39
Warning
Use of Traversable.head
com.sksamuel.scapegoat.inspections.collections.TraversableHead
nodeSeq.head
.home.travis.build.ZGIS.smart-portal-backend.app.models.sosdata.Wml2Export.scala:129
Warning
Use of Traversable.head
com.sksamuel.scapegoat.inspections.collections.TraversableHead
nodeSeq.head
.home.travis.build.ZGIS.smart-portal-backend.app.models.tvp.TimeValuePair.scala:36
Info
Redundant final modifier on case class
com.sksamuel.scapegoat.inspections.RedundantFinalModifierOnCaseClass
Case classes cannot be extended, final modifer is redundant
.home.travis.build.ZGIS.smart-portal-backend.app.models.tvp.XmlTvpParser.scala:89
Warning
Use of asInstanceOf
com.sksamuel.scapegoat.inspections.unsafe.AsInstanceOf
asInstanceOf used near eventReader.next().asInstanceOf[scala.xml.pull.EvText]. Consider using pattern matching.
.home.travis.build.ZGIS.smart-portal-backend.app.models.tvp.XmlTvpParser.scala:103
Warning
Use of asInstanceOf
com.sksamuel.scapegoat.inspections.unsafe.AsInstanceOf
asInstanceOf used near eventReader.next().asInstanceOf[scala.xml.pull.EvText]. Consider using pattern matching.
.home.travis.build.ZGIS.smart-portal-backend.app.models.tvp.XmlTvpParser.scala:114
Warning
Use of asInstanceOf
com.sksamuel.scapegoat.inspections.unsafe.AsInstanceOf
asInstanceOf used near eventReader.next().asInstanceOf[scala.xml.pull.EvText]. Consider using pattern matching.
.home.travis.build.ZGIS.smart-portal-backend.app.models.tvp.XmlTvpParser.scala:119
Warning
Use of asInstanceOf
com.sksamuel.scapegoat.inspections.unsafe.AsInstanceOf
asInstanceOf used near eventReader.next().asInstanceOf[scala.xml.pull.EvText]. Consider using pattern matching.
.home.travis.build.ZGIS.smart-portal-backend.app.models.tvp.XmlTvpParser.scala:216
Warning
Use of asInstanceOf
com.sksamuel.scapegoat.inspections.unsafe.AsInstanceOf
asInstanceOf used near eventReader.next().asInstanceOf[scala.xml.pull.EvText]. Consider using pattern matching.
.home.travis.build.ZGIS.smart-portal-backend.app.models.tvp.XmlTvpParser.scala:236
Warning
Use of asInstanceOf
com.sksamuel.scapegoat.inspections.unsafe.AsInstanceOf
asInstanceOf used near eventReader.next().asInstanceOf[scala.xml.pull.EvText]. Consider using pattern matching.
.home.travis.build.ZGIS.smart-portal-backend.app.models.tvp.XmlTvpParser.scala:248
Warning
Use of asInstanceOf
com.sksamuel.scapegoat.inspections.unsafe.AsInstanceOf
asInstanceOf used near eventReader.next().asInstanceOf[scala.xml.pull.EvText]. Consider using pattern matching.
.home.travis.build.ZGIS.smart-portal-backend.app.models.tvp.XmlTvpParser.scala:253
Warning
Use of asInstanceOf
com.sksamuel.scapegoat.inspections.unsafe.AsInstanceOf
asInstanceOf used near eventReader.next().asInstanceOf[scala.xml.pull.EvText]. Consider using pattern matching.
.home.travis.build.ZGIS.smart-portal-backend.app.models.tvp.XmlTvpParser.scala:60
Warning
Null assignment
com.sksamuel.scapegoat.inspections.nulls.NullAssignment
Null assignment on line 60
.home.travis.build.ZGIS.smart-portal-backend.app.models.tvp.XmlTvpParser.scala:62
Warning
Null assignment
com.sksamuel.scapegoat.inspections.nulls.NullAssignment
Null assignment on line 62
.home.travis.build.ZGIS.smart-portal-backend.app.models.tvp.XmlTvpParser.scala:64
Warning
Null assignment
com.sksamuel.scapegoat.inspections.nulls.NullAssignment
Null assignment on line 64
.home.travis.build.ZGIS.smart-portal-backend.app.models.tvp.XmlTvpParser.scala:66
Warning
Null assignment
com.sksamuel.scapegoat.inspections.nulls.NullAssignment
Null assignment on line 66
.home.travis.build.ZGIS.smart-portal-backend.app.models.tvp.XmlTvpParser.scala:70
Warning
Null assignment
com.sksamuel.scapegoat.inspections.nulls.NullAssignment
Null assignment on line 70
.home.travis.build.ZGIS.smart-portal-backend.app.models.tvp.XmlTvpParser.scala:72
Warning
Null assignment
com.sksamuel.scapegoat.inspections.nulls.NullAssignment
Null assignment on line 72
.home.travis.build.ZGIS.smart-portal-backend.app.models.tvp.XmlTvpParser.scala:187
Warning
Null assignment
com.sksamuel.scapegoat.inspections.nulls.NullAssignment
Null assignment on line 187
.home.travis.build.ZGIS.smart-portal-backend.app.models.tvp.XmlTvpParser.scala:189
Warning
Null assignment
com.sksamuel.scapegoat.inspections.nulls.NullAssignment
Null assignment on line 189
.home.travis.build.ZGIS.smart-portal-backend.app.models.tvp.XmlTvpParser.scala:191
Warning
Null assignment
com.sksamuel.scapegoat.inspections.nulls.NullAssignment
Null assignment on line 191
.home.travis.build.ZGIS.smart-portal-backend.app.models.tvp.XmlTvpParser.scala:193
Warning
Null assignment
com.sksamuel.scapegoat.inspections.nulls.NullAssignment
Null assignment on line 193
.home.travis.build.ZGIS.smart-portal-backend.app.models.tvp.XmlTvpParser.scala:197
Warning
Null assignment
com.sksamuel.scapegoat.inspections.nulls.NullAssignment
Null assignment on line 197
.home.travis.build.ZGIS.smart-portal-backend.app.models.tvp.XmlTvpParser.scala:199
Warning
Null assignment
com.sksamuel.scapegoat.inspections.nulls.NullAssignment
Null assignment on line 199
.home.travis.build.ZGIS.smart-portal-backend.app.models.tvp.XmlTvpParser.scala:76
Info
Match instead of partial function
com.sksamuel.scapegoat.inspections.matching.PartialFunctionInsteadOfMatch
A map match can be replaced with a partial function for greater readability: eventReader.foreach[Unit](((event: scala.xml.pull.XMLEvent) => event match {
case (pre: String, label: String, attrs: scala.xml.MetaData, scope: scala.xml.NamespaceBinding)scala.xml.pull.EvElemStart((prefix @ _), (label @ _), (attributes @ _), (ns @ _)) => {
if (label.==(XmlTvpParser.this.OMOBS))
{
XmlTvpParser.this.logger.trace(scala.StringContext.apply("", ":", " start").s(prefix, label));
obsProp = "";
foiId = "";
geom = scala.None;
measUnit
.home.travis.build.ZGIS.smart-portal-backend.app.models.tvp.XmlTvpParser.scala:203
Info
Match instead of partial function
com.sksamuel.scapegoat.inspections.matching.PartialFunctionInsteadOfMatch
A map match can be replaced with a partial function for greater readability: eventReader.foreach[Unit](((event: scala.xml.pull.XMLEvent) => event match {
case (pre: String, label: String, attrs: scala.xml.MetaData, scope: scala.xml.NamespaceBinding)scala.xml.pull.EvElemStart((prefix @ _), (label @ _), (attributes @ _), (ns @ _)) => {
if (label.==(XmlTvpParser.this.OMOBS))
{
XmlTvpParser.this.logger.trace(scala.StringContext.apply("", ":", " start").s(prefix, label));
obsProp = "";
foiId = "";
geom = scala.None;
measUnit
.home.travis.build.ZGIS.smart-portal-backend.app.models.users.User.scala:41
Info
Redundant final modifier on case class
com.sksamuel.scapegoat.inspections.RedundantFinalModifierOnCaseClass
Case classes cannot be extended, final modifer is redundant
.home.travis.build.ZGIS.smart-portal-backend.app.models.users.User.scala:100
Info
Redundant final modifier on case class
com.sksamuel.scapegoat.inspections.RedundantFinalModifierOnCaseClass
Case classes cannot be extended, final modifer is redundant
.home.travis.build.ZGIS.smart-portal-backend.app.models.users.User.scala:102
Info
Redundant final modifier on case class
com.sksamuel.scapegoat.inspections.RedundantFinalModifierOnCaseClass
Case classes cannot be extended, final modifer is redundant
.home.travis.build.ZGIS.smart-portal-backend.app.models.users.User.scala:109
Info
Redundant final modifier on case class
com.sksamuel.scapegoat.inspections.RedundantFinalModifierOnCaseClass
Case classes cannot be extended, final modifer is redundant
.home.travis.build.ZGIS.smart-portal-backend.app.models.users.UserDAO.scala:147
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
u.get
.home.travis.build.ZGIS.smart-portal-backend.app.models.users.UserFile.scala:44
Info
Redundant final modifier on case class
com.sksamuel.scapegoat.inspections.RedundantFinalModifierOnCaseClass
Case classes cannot be extended, final modifer is redundant
.home.travis.build.ZGIS.smart-portal-backend.app.models.users.UserFile.scala:121
Warning
Unused method parameter
com.sksamuel.scapegoat.inspections.unneccesary.UnusedMethodParameter
Unused method parameter (val userFile: models.users.UserFile = _)
.home.travis.build.ZGIS.smart-portal-backend.app.models.users.UserFile.scala:121
Warning
Unused method parameter
com.sksamuel.scapegoat.inspections.unneccesary.UnusedMethodParameter
Unused method parameter (implicit val connection: java.sql.Connection = _)
.home.travis.build.ZGIS.smart-portal-backend.app.models.users.UserGroup.scala:70
Info
List.size is O(n)
com.sksamuel.scapegoat.inspections.collections.ListSize
List.size is O(n). Consider using a different data type with O(1) size lookup such as Vector or Array.
.home.travis.build.ZGIS.smart-portal-backend.app.models.users.UserGroup.scala:85
Info
List.size is O(n)
com.sksamuel.scapegoat.inspections.collections.ListSize
List.size is O(n). Consider using a different data type with O(1) size lookup such as Vector or Array.
.home.travis.build.ZGIS.smart-portal-backend.app.models.users.UserGroup.scala:120
Info
List.size is O(n)
com.sksamuel.scapegoat.inspections.collections.ListSize
List.size is O(n). Consider using a different data type with O(1) size lookup such as Vector or Array.
.home.travis.build.ZGIS.smart-portal-backend.app.models.users.UserGroup.scala:120
Info
List.size is O(n)
com.sksamuel.scapegoat.inspections.collections.ListSize
List.size is O(n). Consider using a different data type with O(1) size lookup such as Vector or Array.
.home.travis.build.ZGIS.smart-portal-backend.app.models.users.UserGroup.scala:127
Info
List.size is O(n)
com.sksamuel.scapegoat.inspections.collections.ListSize
List.size is O(n). Consider using a different data type with O(1) size lookup such as Vector or Array.
.home.travis.build.ZGIS.smart-portal-backend.app.models.users.UserGroup.scala:128
Info
List.size is O(n)
com.sksamuel.scapegoat.inspections.collections.ListSize
List.size is O(n). Consider using a different data type with O(1) size lookup such as Vector or Array.
.home.travis.build.ZGIS.smart-portal-backend.app.models.users.UserGroup.scala:274
Info
List.size is O(n)
com.sksamuel.scapegoat.inspections.collections.ListSize
List.size is O(n). Consider using a different data type with O(1) size lookup such as Vector or Array.
.home.travis.build.ZGIS.smart-portal-backend.app.models.users.UserGroup.scala:274
Info
List.size is O(n)
com.sksamuel.scapegoat.inspections.collections.ListSize
List.size is O(n). Consider using a different data type with O(1) size lookup such as Vector or Array.
.home.travis.build.ZGIS.smart-portal-backend.app.models.users.UserGroup.scala:280
Info
List.size is O(n)
com.sksamuel.scapegoat.inspections.collections.ListSize
List.size is O(n). Consider using a different data type with O(1) size lookup such as Vector or Array.
.home.travis.build.ZGIS.smart-portal-backend.app.models.users.UserGroup.scala:281
Info
List.size is O(n)
com.sksamuel.scapegoat.inspections.collections.ListSize
List.size is O(n). Consider using a different data type with O(1) size lookup such as Vector or Array.
.home.travis.build.ZGIS.smart-portal-backend.app.models.users.UserGroup.scala:32
Info
Redundant final modifier on case class
com.sksamuel.scapegoat.inspections.RedundantFinalModifierOnCaseClass
Case classes cannot be extended, final modifer is redundant
.home.travis.build.ZGIS.smart-portal-backend.app.models.users.UserGroup.scala:42
Info
Redundant final modifier on case class
com.sksamuel.scapegoat.inspections.RedundantFinalModifierOnCaseClass
Case classes cannot be extended, final modifer is redundant
.home.travis.build.ZGIS.smart-portal-backend.app.models.users.UserGroup.scala:44
Info
Redundant final modifier on case class
com.sksamuel.scapegoat.inspections.RedundantFinalModifierOnCaseClass
Case classes cannot be extended, final modifer is redundant
.home.travis.build.ZGIS.smart-portal-backend.app.models.users.UserGroupContextsVisibility.scala:29
Info
Redundant final modifier on case class
com.sksamuel.scapegoat.inspections.RedundantFinalModifierOnCaseClass
Case classes cannot be extended, final modifer is redundant
.home.travis.build.ZGIS.smart-portal-backend.app.models.users.UserGroupUsersLevel.scala:36
Info
Redundant final modifier on case class
com.sksamuel.scapegoat.inspections.RedundantFinalModifierOnCaseClass
Case classes cannot be extended, final modifer is redundant
.home.travis.build.ZGIS.smart-portal-backend.app.models.users.UserLinkLogging.scala:53
Info
Redundant final modifier on case class
com.sksamuel.scapegoat.inspections.RedundantFinalModifierOnCaseClass
Case classes cannot be extended, final modifer is redundant
.home.travis.build.ZGIS.smart-portal-backend.app.models.users.UserMetaRecord.scala:43
Info
Redundant final modifier on case class
com.sksamuel.scapegoat.inspections.RedundantFinalModifierOnCaseClass
Case classes cannot be extended, final modifer is redundant
.home.travis.build.ZGIS.smart-portal-backend.app.models.users.UserMetaRecord.scala:121
Warning
Unused method parameter
com.sksamuel.scapegoat.inspections.unneccesary.UnusedMethodParameter
Unused method parameter (val userMetaRecord: models.users.UserMetaRecord = _)
.home.travis.build.ZGIS.smart-portal-backend.app.models.users.UserMetaRecord.scala:121
Warning
Unused method parameter
com.sksamuel.scapegoat.inspections.unneccesary.UnusedMethodParameter
Unused method parameter (implicit val connection: java.sql.Connection = _)
.home.travis.build.ZGIS.smart-portal-backend.app.models.users.UserSession.scala:49
Info
Redundant final modifier on case class
com.sksamuel.scapegoat.inspections.RedundantFinalModifierOnCaseClass
Case classes cannot be extended, final modifer is redundant
.home.travis.build.ZGIS.smart-portal-backend.app.services.AdminService.scala:48
Info
Unnecessary if condition.
com.sksamuel.scapegoat.inspections.unneccesary.UnnecessaryIf
If comparison is not needed. Use the condition. Eg, instead of if (a ==b) true else false, simply use a == b. : if (adminList.contains(email))
true
else
false
.home.travis.build.ZGIS.smart-portal-backend.app.services.AdminService.scala:35
Warning
Unused constructor parameter
com.sksamuel.scapegoat.inspections.unneccesary.UnusedMethodParameter
Unused constructor parameter (cache)
.home.travis.build.ZGIS.smart-portal-backend.app.services.EmailService.scala:76
Warning
Catch exception
com.sksamuel.scapegoat.inspections.exception.CatchException
Did you intend to catch all exceptions, consider catching a more specific exception class: try {
val request: com.sendgrid.Request = new com.sendgrid.Request();
request.method = POST;
request.endpoint = "mail/send";
request.body = mail.build();
val response: com.sendgrid.Response = EmailService.this.sg.api(request);
EmailService.this.logger.trace(scala.StringContext.apply("mai
.home.travis.build.ZGIS.smart-portal-backend.app.services.EmailService.scala:132
Warning
Catch exception
com.sksamuel.scapegoat.inspections.exception.CatchException
Did you intend to catch all exceptions, consider catching a more specific exception class: try {
val request: com.sendgrid.Request = new com.sendgrid.Request();
request.method = POST;
request.endpoint = "mail/send";
request.body = mail.build();
val response: com.sendgrid.Response = EmailService.this.sg.api(request);
EmailService.this.logger.trace(scala.StringContext.apply("mai
.home.travis.build.ZGIS.smart-portal-backend.app.services.EmailService.scala:187
Warning
Catch exception
com.sksamuel.scapegoat.inspections.exception.CatchException
Did you intend to catch all exceptions, consider catching a more specific exception class: try {
val request: com.sendgrid.Request = new com.sendgrid.Request();
request.method = POST;
request.endpoint = "mail/send";
request.body = mail.build();
val response: com.sendgrid.Response = EmailService.this.sg.api(request);
EmailService.this.logger.trace(scala.StringContext.apply("mai
.home.travis.build.ZGIS.smart-portal-backend.app.services.EmailService.scala:243
Warning
Catch exception
com.sksamuel.scapegoat.inspections.exception.CatchException
Did you intend to catch all exceptions, consider catching a more specific exception class: try {
val request: com.sendgrid.Request = new com.sendgrid.Request();
request.method = POST;
request.endpoint = "mail/send";
request.body = mail.build();
val response: com.sendgrid.Response = EmailService.this.sg.api(request);
EmailService.this.logger.trace(scala.StringContext.apply("mai
.home.travis.build.ZGIS.smart-portal-backend.app.services.EmailService.scala:300
Warning
Catch exception
com.sksamuel.scapegoat.inspections.exception.CatchException
Did you intend to catch all exceptions, consider catching a more specific exception class: try {
val request: com.sendgrid.Request = new com.sendgrid.Request();
request.method = POST;
request.endpoint = "mail/send";
request.body = mail.build();
val response: com.sendgrid.Response = EmailService.this.sg.api(request);
EmailService.this.logger.trace(scala.StringContext.apply("mai
.home.travis.build.ZGIS.smart-portal-backend.app.services.EmailService.scala:356
Warning
Catch exception
com.sksamuel.scapegoat.inspections.exception.CatchException
Did you intend to catch all exceptions, consider catching a more specific exception class: try {
val request: com.sendgrid.Request = new com.sendgrid.Request();
request.method = POST;
request.endpoint = "mail/send";
request.body = mail.build();
val response: com.sendgrid.Response = EmailService.this.sg.api(request);
EmailService.this.logger.trace(scala.StringContext.apply("mai
.home.travis.build.ZGIS.smart-portal-backend.app.services.EmailService.scala:411
Warning
Catch exception
com.sksamuel.scapegoat.inspections.exception.CatchException
Did you intend to catch all exceptions, consider catching a more specific exception class: try {
val request: com.sendgrid.Request = new com.sendgrid.Request();
request.method = POST;
request.endpoint = "mail/send";
request.body = mail.build();
val response: com.sendgrid.Response = EmailService.this.sg.api(request);
EmailService.this.logger.trace(scala.StringContext.apply("mai
.home.travis.build.ZGIS.smart-portal-backend.app.services.GoogleServicesDAO.scala:232
Warning
Null parameter
com.sksamuel.scapegoat.inspections.nulls.NullParameter
Null is used as a method parameter: blob.!=(null)
.home.travis.build.ZGIS.smart-portal-backend.app.services.GoogleServicesDAO.scala:233
Warning
Null parameter
com.sksamuel.scapegoat.inspections.nulls.NullParameter
Null is used as a method parameter: blob.==(null)
.home.travis.build.ZGIS.smart-portal-backend.app.services.GoogleServicesDAO.scala:41
Info
Redundant final modifier on case class
com.sksamuel.scapegoat.inspections.RedundantFinalModifierOnCaseClass
Case classes cannot be extended, final modifer is redundant
.home.travis.build.ZGIS.smart-portal-backend.app.services.MetadataService.scala:63
Warning
Java conversions
com.sksamuel.scapegoat.inspections.collections.JavaConversionsUse
Use of java conversions can lead to unusual behaviour. It is recommended to use JavaConverters
.home.travis.build.ZGIS.smart-portal-backend.app.services.MetadataService.scala:41
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
ValidValuesReadsAdditions.this.metadataService.getValidValuesFor(topic).get
.home.travis.build.ZGIS.smart-portal-backend.app.services.MetadataService.scala:66
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
MetadataService.this.metadataValidValues.get
.home.travis.build.ZGIS.smart-portal-backend.app.services.MetadataService.scala:73
Warning
Use of Traversable.head
com.sksamuel.scapegoat.inspections.collections.TraversableHead
validValuesConf.head
.home.travis.build.ZGIS.smart-portal-backend.app.services.MetadataService.scala:74
Warning
Use of Traversable.head
com.sksamuel.scapegoat.inspections.collections.TraversableHead
validValuesConf.head
.home.travis.build.ZGIS.smart-portal-backend.app.services.MetadataService.scala:76
Warning
Use of Traversable.head
com.sksamuel.scapegoat.inspections.collections.TraversableHead
validValuesConf.head
.home.travis.build.ZGIS.smart-portal-backend.app.services.OwcCollectionsService.scala:479
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
owcAuthor.get
.home.travis.build.ZGIS.smart-portal-backend.app.services.OwcCollectionsService.scala:75
Info
Prefer Seq.empty
com.sksamuel.scapegoat.inspections.collections.PreferSeqEmpty
Seq[T]() creates a new instance. Consider Seq.empty which does not allocate a new object. collection.this.Seq.apply[info.smart.models.owc100.OwcLink]()
.home.travis.build.ZGIS.smart-portal-backend.app.services.OwcCollectionsService.scala:273
Warning
Unused method parameter
com.sksamuel.scapegoat.inspections.unneccesary.UnusedMethodParameter
Unused method parameter (val externalCatalogUrl: String = _)
.home.travis.build.ZGIS.smart-portal-backend.app.services.OwcCollectionsService.scala:353
Warning
Unused method parameter
com.sksamuel.scapegoat.inspections.unneccesary.UnusedMethodParameter
Unused method parameter (val externalCatalogUrl: String = _)
.home.travis.build.ZGIS.smart-portal-backend.app.services.OwcCollectionsService.scala:396
Warning
Unused method parameter
com.sksamuel.scapegoat.inspections.unneccesary.UnusedMethodParameter
Unused method parameter (val externalCatalogUrl: String = _)
.home.travis.build.ZGIS.smart-portal-backend.app.services.OwcCollectionsService.scala:622
Warning
Var could be val
com.sksamuel.scapegoat.inspections.unneccesary.VarCouldBeVal
bboxCalc is never written to, so could be a val: def calculateBBoxForCollection(owcContext: info.smart.models.owc100.OwcContext): Option[org.locationtech.spatial4j.shape.Rectangle] = {
def computeJointRect(leftRo: Option[org.locationtech.spatial4j
.home.travis.build.ZGIS.smart-portal-backend.app.services.PortalConfig.scala:115
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
PortalConfig.this.getOrReportStr("datetime.timezone").get
.home.travis.build.ZGIS.smart-portal-backend.app.services.PortalConfig.scala:122
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
PortalConfig.this.getOrReportStr("email.sendgrid.apikey").get
.home.travis.build.ZGIS.smart-portal-backend.app.services.PortalConfig.scala:123
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
PortalConfig.this.getOrReportStr("email.sendgrid.from").get
.home.travis.build.ZGIS.smart-portal-backend.app.services.PortalConfig.scala:124
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
PortalConfig.this.getOrReportStr("email.sendgrid.replyto").get
.home.travis.build.ZGIS.smart-portal-backend.app.services.PortalConfig.scala:130
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
PortalConfig.this.getOrReportStr("smart.upload.datapath").get
.home.travis.build.ZGIS.smart-portal-backend.app.services.PortalConfig.scala:135
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
PortalConfig.this.getOrReportStr("play.crypto.secret").get
.home.travis.build.ZGIS.smart-portal-backend.app.services.PortalConfig.scala:141
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
PortalConfig.this.getOrReportStr("smart.csw.url").get
.home.travis.build.ZGIS.smart-portal-backend.app.services.PortalConfig.scala:154
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
PortalConfig.this.getOrReportStr("smart.base.url").get
.home.travis.build.ZGIS.smart-portal-backend.app.services.PortalConfig.scala:160
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
PortalConfig.this.getOrReportStr("smart.csw-ingester.url").get
.home.travis.build.ZGIS.smart-portal-backend.app.services.PortalConfig.scala:167
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
PortalConfig.this.getOrReportStr("smart.vocab.url").get
.home.travis.build.ZGIS.smart-portal-backend.app.services.PortalConfig.scala:174
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
PortalConfig.this.getOrReportStr("smart.admin.url").get
.home.travis.build.ZGIS.smart-portal-backend.app.services.PortalConfig.scala:186
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
PortalConfig.this.getOrReportStr("google.recaptcha.secret").get
.home.travis.build.ZGIS.smart-portal-backend.app.services.PortalConfig.scala:196
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
PortalConfig.this.getOrReportStr("google.client.secret").get
.home.travis.build.ZGIS.smart-portal-backend.app.services.PortalConfig.scala:201
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
PortalConfig.this.getOrReportStr("google.storage.bucket").get
.home.travis.build.ZGIS.smart-portal-backend.app.services.PortalConfig.scala:206
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
PortalConfig.this.getOrReportStr("google.project.id").get
.home.travis.build.ZGIS.smart-portal-backend.app.services.PortalConfig.scala:91
Info
Match instead of partial function
com.sksamuel.scapegoat.inspections.matching.PartialFunctionInsteadOfMatch
A map match can be replaced with a partial function for greater readability: a.fold[Option[A]](throw new scala.`package`.NoSuchElementException("Config not found for ".+(path)))(((lst: A) => lst match {
case (x @ (_: PortalConfig.this.JListConfiguration)) => {
if (x.size().<=(0))
PortalConfig.this.logger.error("Config is empty list for ".+(path))
else
();
scala.Some.apply[A](lst)
}
case (x @ (_: PortalConfig.this.JListString)) => {
if (x.size().<=(0))
PortalConfig.this.logger.error("Config is empty list for ".+(path))
else
.home.travis.build.ZGIS.smart-portal-backend.app.services.UserGroupService.scala:141
Info
Prefer Seq.empty
com.sksamuel.scapegoat.inspections.collections.PreferSeqEmpty
Seq[T]() creates a new instance. Consider Seq.empty which does not allocate a new object. collection.this.Seq.apply[Nothing]()
.home.travis.build.ZGIS.smart-portal-backend.app.services.UserGroupService.scala:84
Warning
Unused method parameter
com.sksamuel.scapegoat.inspections.unneccesary.UnusedMethodParameter
Unused method parameter (val user: models.users.User = _)
.home.travis.build.ZGIS.smart-portal-backend.app.services.UserService.scala:54
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
userOpt.get
.home.travis.build.ZGIS.smart-portal-backend.app.services.UserService.scala:283
Warning
Unused method parameter
com.sksamuel.scapegoat.inspections.unneccesary.UnusedMethodParameter
Unused method parameter (val userEmail: String = _)
.home.travis.build.ZGIS.smart-portal-backend.app.utils.GeoDateParserUtils.scala:49
Warning
Use of asInstanceOf
com.sksamuel.scapegoat.inspections.unsafe.AsInstanceOf
asInstanceOf used near GeoDateParserUtils.this.wktReader.read(bboxAsWkt.get).asInstanceOf[org.locationtech.spatial4j.shape.Rectangle]. Consider using pattern matching.
.home.travis.build.ZGIS.smart-portal-backend.app.utils.GeoDateParserUtils.scala:49
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
bboxAsWkt.get
.home.travis.build.ZGIS.smart-portal-backend.app.utils.GeoDateParserUtils.scala:74
Warning
Use of Option.get
com.sksamuel.scapegoat.inspections.option.OptionGet
dateStringOption.get
.home.travis.build.ZGIS.smart-portal-backend.app.utils.GeoDateParserUtils.scala:111
Warning
Use of Traversable.head
com.sksamuel.scapegoat.inspections.collections.TraversableHead
dateStrings.head
.home.travis.build.ZGIS.smart-portal-backend.app.utils.GeoDateParserUtils.scala:126
Warning
Use of Traversable.head
com.sksamuel.scapegoat.inspections.collections.TraversableHead
dates.head
.home.travis.build.ZGIS.smart-portal-backend.app.utils.GeoDateParserUtils.scala:127
Warning
Use of Traversable.head
com.sksamuel.scapegoat.inspections.collections.TraversableHead
dates.head
.home.travis.build.ZGIS.smart-portal-backend.app.utils.GeoDateParserUtils.scala:130
Warning
Use of Traversable.head
com.sksamuel.scapegoat.inspections.collections.TraversableHead
dates.head
.home.travis.build.ZGIS.smart-portal-backend.app.utils.OwcGeoJsonFixes.scala:48
Warning
Use of asInstanceOf
com.sksamuel.scapegoat.inspections.unsafe.AsInstanceOf
asInstanceOf used near {
<artifact> val x$1: List[info.smart.models.owc100.OwcLink] @scala.reflect.internal.annotations.uncheckedBounds = specReference;
<artifact> val x$2: List[info.smart.models.owc100.OwcLink] @scala.reflect.internal.annotations.uncheckedBounds = contextMetadata;
<artifact> val x$3: java.net.URL = owcUnfixed.copy$default$1;
<artifact> val x$4: Option[org.locationtech.spatial4j.shape.Rectangle] @scala.reflect.internal.annotations.uncheckedBounds = owcUnfixed.copy$default$2;
<artifact> val x. Consider using pattern matching.
.home.travis.build.ZGIS.smart-portal-backend.app.utils.OwcGeoJsonFixes.scala:56
Warning
Use of asInstanceOf
com.sksamuel.scapegoat.inspections.unsafe.AsInstanceOf
asInstanceOf used near {
<artifact> val x$17: List[info.smart.models.owc100.OwcLink] @scala.reflect.internal.annotations.uncheckedBounds = contentDescription;
<artifact> val x$18: List[info.smart.models.owc100.OwcLink] @scala.reflect.internal.annotations.uncheckedBounds = preview;
<artifact> val x$19: List[info.smart.models.owc100.OwcLink] @scala.reflect.internal.annotations.uncheckedBounds = contentByRef;
<artifact> val x$20: List[info.smart.models.owc100.OwcLink] @scala.reflect.internal.annotations.unchecked. Consider using pattern matching.
.home.travis.build.ZGIS.smart-portal-backend.target.scala-2.11.routes.main.controllers.ReverseRoutes.scala:17
Info
Method name not recommended
com.sksamuel.scapegoat.inspections.naming.MethodNames
Methods should be in camelCase style with the first letter lower-case. See http://docs.scala-lang.org/style/naming-conventions.html#methods
.home.travis.build.ZGIS.smart-portal-backend.target.scala-2.11.routes.main.controllers.ReverseRoutes.scala:32
Info
Method name not recommended
com.sksamuel.scapegoat.inspections.naming.MethodNames
Methods should be in camelCase style with the first letter lower-case. See http://docs.scala-lang.org/style/naming-conventions.html#methods
.home.travis.build.ZGIS.smart-portal-backend.target.scala-2.11.routes.main.controllers.ReverseRoutes.scala:131
Info
Method name not recommended
com.sksamuel.scapegoat.inspections.naming.MethodNames
Methods should be in camelCase style with the first letter lower-case. See http://docs.scala-lang.org/style/naming-conventions.html#methods
.home.travis.build.ZGIS.smart-portal-backend.target.scala-2.11.routes.main.controllers.ReverseRoutes.scala:224
Info
Method name not recommended
com.sksamuel.scapegoat.inspections.naming.MethodNames
Methods should be in camelCase style with the first letter lower-case. See http://docs.scala-lang.org/style/naming-conventions.html#methods
.home.travis.build.ZGIS.smart-portal-backend.target.scala-2.11.routes.main.controllers.ReverseRoutes.scala:269
Info
Method name not recommended
com.sksamuel.scapegoat.inspections.naming.MethodNames
Methods should be in camelCase style with the first letter lower-case. See http://docs.scala-lang.org/style/naming-conventions.html#methods
.home.travis.build.ZGIS.smart-portal-backend.target.scala-2.11.routes.main.controllers.ReverseRoutes.scala:326
Info
Method name not recommended
com.sksamuel.scapegoat.inspections.naming.MethodNames
Methods should be in camelCase style with the first letter lower-case. See http://docs.scala-lang.org/style/naming-conventions.html#methods
.home.travis.build.ZGIS.smart-portal-backend.target.scala-2.11.routes.main.controllers.ReverseRoutes.scala:421
Info
Method name not recommended
com.sksamuel.scapegoat.inspections.naming.MethodNames
Methods should be in camelCase style with the first letter lower-case. See http://docs.scala-lang.org/style/naming-conventions.html#methods
.home.travis.build.ZGIS.smart-portal-backend.target.scala-2.11.routes.main.controllers.ReverseRoutes.scala:454
Info
Method name not recommended
com.sksamuel.scapegoat.inspections.naming.MethodNames
Methods should be in camelCase style with the first letter lower-case. See http://docs.scala-lang.org/style/naming-conventions.html#methods
.home.travis.build.ZGIS.smart-portal-backend.target.scala-2.11.routes.main.controllers.ReverseRoutes.scala:481
Info
Method name not recommended
com.sksamuel.scapegoat.inspections.naming.MethodNames
Methods should be in camelCase style with the first letter lower-case. See http://docs.scala-lang.org/style/naming-conventions.html#methods
.home.travis.build.ZGIS.smart-portal-backend.target.scala-2.11.routes.main.controllers.ReverseRoutes.scala:161
Info
Prefer Seq.empty
com.sksamuel.scapegoat.inspections.collections.PreferSeqEmpty
Seq[T]() creates a new instance. Consider Seq.empty which does not allocate a new object. collection.this.Seq.apply[Nothing]()
.home.travis.build.ZGIS.smart-portal-backend.target.scala-2.11.routes.main.controllers.ReverseRoutes.scala:163
Info
Prefer Seq.empty
com.sksamuel.scapegoat.inspections.collections.PreferSeqEmpty
Seq[T]() creates a new instance. Consider Seq.empty which does not allocate a new object. collection.this.Seq.apply[Nothing]()
.home.travis.build.ZGIS.smart-portal-backend.target.scala-2.11.routes.main.controllers.javascript.JavaScriptReverseRoutes.scala:20
Info
Method name not recommended
com.sksamuel.scapegoat.inspections.naming.MethodNames
Methods should be in camelCase style with the first letter lower-case. See http://docs.scala-lang.org/style/naming-conventions.html#methods
.home.travis.build.ZGIS.smart-portal-backend.target.scala-2.11.routes.main.controllers.javascript.JavaScriptReverseRoutes.scala:40
Info
Method name not recommended
com.sksamuel.scapegoat.inspections.naming.MethodNames
Methods should be in camelCase style with the first letter lower-case. See http://docs.scala-lang.org/style/naming-conventions.html#methods
.home.travis.build.ZGIS.smart-portal-backend.target.scala-2.11.routes.main.controllers.javascript.JavaScriptReverseRoutes.scala:200
Info
Method name not recommended
com.sksamuel.scapegoat.inspections.naming.MethodNames
Methods should be in camelCase style with the first letter lower-case. See http://docs.scala-lang.org/style/naming-conventions.html#methods
.home.travis.build.ZGIS.smart-portal-backend.target.scala-2.11.routes.main.controllers.javascript.JavaScriptReverseRoutes.scala:350
Info
Method name not recommended
com.sksamuel.scapegoat.inspections.naming.MethodNames
Methods should be in camelCase style with the first letter lower-case. See http://docs.scala-lang.org/style/naming-conventions.html#methods
.home.travis.build.ZGIS.smart-portal-backend.target.scala-2.11.routes.main.controllers.javascript.JavaScriptReverseRoutes.scala:420
Info
Method name not recommended
com.sksamuel.scapegoat.inspections.naming.MethodNames
Methods should be in camelCase style with the first letter lower-case. See http://docs.scala-lang.org/style/naming-conventions.html#methods
.home.travis.build.ZGIS.smart-portal-backend.target.scala-2.11.routes.main.controllers.javascript.JavaScriptReverseRoutes.scala:510
Info
Method name not recommended
com.sksamuel.scapegoat.inspections.naming.MethodNames
Methods should be in camelCase style with the first letter lower-case. See http://docs.scala-lang.org/style/naming-conventions.html#methods
.home.travis.build.ZGIS.smart-portal-backend.target.scala-2.11.routes.main.controllers.javascript.JavaScriptReverseRoutes.scala:654
Info
Method name not recommended
com.sksamuel.scapegoat.inspections.naming.MethodNames
Methods should be in camelCase style with the first letter lower-case. See http://docs.scala-lang.org/style/naming-conventions.html#methods
.home.travis.build.ZGIS.smart-portal-backend.target.scala-2.11.routes.main.controllers.javascript.JavaScriptReverseRoutes.scala:704
Info
Method name not recommended
com.sksamuel.scapegoat.inspections.naming.MethodNames
Methods should be in camelCase style with the first letter lower-case. See http://docs.scala-lang.org/style/naming-conventions.html#methods
.home.travis.build.ZGIS.smart-portal-backend.target.scala-2.11.routes.main.controllers.javascript.JavaScriptReverseRoutes.scala:744
Info
Method name not recommended
com.sksamuel.scapegoat.inspections.naming.MethodNames
Methods should be in camelCase style with the first letter lower-case. See http://docs.scala-lang.org/style/naming-conventions.html#methods
.home.travis.build.ZGIS.smart-portal-backend.target.scala-2.11.routes.main.router.Routes.scala:145
Warning
Use of asInstanceOf
com.sksamuel.scapegoat.inspections.unsafe.AsInstanceOf
asInstanceOf used near r.asInstanceOf[(String, String, String)]. Consider using pattern matching.
.home.travis.build.ZGIS.smart-portal-backend.target.scala-2.11.routes.main.router.Routes.scala:146
Warning
Use of asInstanceOf
com.sksamuel.scapegoat.inspections.unsafe.AsInstanceOf
asInstanceOf used near l.asInstanceOf[List[(String, String, String)]]. Consider using pattern matching.
.home.travis.build.ZGIS.smart-portal-backend.target.scala-2.11.routes.main.router.Routes.scala:145
Info
List append is slow
com.sksamuel.scapegoat.inspections.collections.ListAppend
List append is O(n). For large lists, consider using cons (::) or another data structure such as ListBuffer or Vector and converting to a List once built.
.home.travis.build.ZGIS.smart-portal-backend.target.scala-2.11.routes.main.router.Routes.scala:1490
Info
Prefer Seq.empty
com.sksamuel.scapegoat.inspections.collections.PreferSeqEmpty
Seq[T]() creates a new instance. Consider Seq.empty which does not allocate a new object. collection.this.Seq.apply[Nothing]()