Scapegoat Inspections

Unused method parameter (val fields: Option[String] = _)

fromDateStr.get

toDateStr.get

query.get

Unused method parameter (val origin: String = _)

asInstanceOf used near MdMetadataSet.this.wktReader.read(envelope).asInstanceOf[org.locationtech.spatial4j.shape.Rectangle]. Consider using pattern matching.

Did you intend to catch all exceptions, consider catching a more specific exception class: try { nodeSeq.head.label match { case "MD_Metadata" => { val identificationInfo: String = "_"; scala.Some.apply[models.gmd.MdMetadataSet]({ <artifact> val x$7: String = nodeSeq.\("fileIdentifier").\("CharacterString").text; <artifact> val x$8: java.time.LocalDate =

List.size is O(n). Consider using a different data type with O(1) size lookup such as Vector or Array.

List.size is O(n). Consider using a different data type with O(1) size lookup such as Vector or Array.

List.size is O(n). Consider using a different data type with O(1) size lookup such as Vector or Array.

List.size is O(n). Consider using a different data type with O(1) size lookup such as Vector or Array.

List.size is O(n). Consider using a different data type with O(1) size lookup such as Vector or Array.

x$3.get

json.this.JsValue.jsValueToJsLookup(bbox).apply(0).asOpt[Double](json.this.Reads.DoubleReads).get

json.this.JsValue.jsValueToJsLookup(bbox).apply(0).asOpt[Double](json.this.Reads.DoubleReads).get

json.this.JsValue.jsValueToJsLookup(bbox).apply(2).asOpt[Double](json.this.Reads.DoubleReads).get

json.this.JsValue.jsValueToJsLookup(bbox).apply(2).asOpt[Double](json.this.Reads.DoubleReads).get

json.this.JsValue.jsValueToJsLookup(bbox).apply(2).asOpt[Double](json.this.Reads.DoubleReads).get

A map match can be replaced with a partial function for greater readability: immutable.this.List.apply[Double](west, east).map[Double, List[Double]](((value: Double) => value match { case (n @ _) if value.>=(MdMetadataSet.this.minLon).&&(value.<=(MdMetadataSet.this.maxLon)) => n case (n @ _) if scala.math.`package`.abs(value.%(scala.math.`package`.abs(MdMetadataSet.this.minLon.-(MdMetadataSet.this.maxLon)))).<(MdMetadataSet.this.maxLon) => { val result: Double = value.%(MdMetadataSet.this.maxLon); MdMetadataSet.this.logger.info(({ val arg$macro$35: Any

nodeSeq.head

nodeSeq.head

datesList.head

Unused method parameter (val nodeSeq: scala.xml.NodeSeq = _)

.filter(_.isDefined).map(_.get) can be replaced with flatten: cswGetRecordsResponse.xml.\\("MD_Metadata").map[Option[models.gmd.MdMetadataSet], scala.collection.immutable.Seq[Option[models.gmd.MdMetadataSet]]](((mdMetadataNode: scala.xml.Node) => { LuceneIndexBuilderActor.this.logger.debug(({ val arg$macro$43: Any = catalogueName; val arg$macro$44: Any = mdMetadataNode.\("fileIdentifier").\("CharacterString").text; new scala.collection.immutable.StringOps("Preparing(%s): %s").format(arg$macro$43, arg$macro$44) }: String)); LuceneIndexBuil

item.get

Unnecessary toString on instanceo of String: wsClientResponse.body.toString

asInstanceOf used near avro4s.this.FromValue.StringFromValue.asInstanceOf[com.sksamuel.avro4s.FromValue.StringFromValue.type]. Consider using pattern matching.

asInstanceOf used near avro4s.this.FromValue.SeqFromValue[String](avro4s.this.FromValue.StringFromValue).asInstanceOf[com.sksamuel.avro4s.FromValue[Seq[String]]]. Consider using pattern matching.

asInstanceOf used near $anon.this.converters.apply(0).asInstanceOf[shapeless.Lazy[com.sksamuel.avro4s.FromValue[String]]]. Consider using pattern matching.

asInstanceOf used near $anon.this.converters.apply(1).asInstanceOf[shapeless.Lazy[com.sksamuel.avro4s.FromValue[Seq[String]]]]. Consider using pattern matching.

asInstanceOf used near avro4s.this.ToValue.StringToValue.asInstanceOf[com.sksamuel.avro4s.ToValue.StringToValue.type]. Consider using pattern matching.

asInstanceOf used near avro4s.this.ToValue.SeqToValue[String](avro4s.this.ToValue.StringToValue).asInstanceOf[com.sksamuel.avro4s.ToValue[Seq[String]]]. Consider using pattern matching.

asInstanceOf used near $anon.this.converters.apply(0).asInstanceOf[shapeless.Lazy[com.sksamuel.avro4s.ToValue[String]]]. Consider using pattern matching.

asInstanceOf used near $anon.this.converters.apply(1).asInstanceOf[shapeless.Lazy[com.sksamuel.avro4s.ToValue[Seq[String]]]]. Consider using pattern matching.

Null is used as a method parameter: new org.apache.lucene.search.SearcherManager(LuceneService.this.indexDirectory, null)

LuceneService.this.configuration.getConfigList("csw.catalogues").get

item.getString("name", item.getString$default$2).get

item.getString("url", item.getString$default$2).get

maxNumberOfResults.get

Seq[T]() creates a new instance. Consider Seq.empty which does not allocate a new object. collection.this.Seq.apply[Nothing]()

Seq[T]() creates a new instance. Consider Seq.empty which does not allocate a new object. collection.this.Seq.apply[Nothing]()

Seq[T]() creates a new instance. Consider Seq.empty which does not allocate a new object. collection.this.Seq.apply[Nothing]()

Seq[T]() creates a new instance. Consider Seq.empty which does not allocate a new object. collection.this.Seq.apply[Nothing]()

Seq[T]() creates a new instance. Consider Seq.empty which does not allocate a new object. collection.this.Seq.apply[Nothing]()

Seq[T]() creates a new instance. Consider Seq.empty which does not allocate a new object. collection.this.Seq.apply[Nothing]()

Seq[T]() creates a new instance. Consider Seq.empty which does not allocate a new object. collection.this.Seq.apply[Nothing]()

Seq[T]() creates a new instance. Consider Seq.empty which does not allocate a new object. collection.this.Seq.apply[Nothing]()

Seq[T]() creates a new instance. Consider Seq.empty which does not allocate a new object. collection.this.Seq.apply[Nothing]()

Unused constructor parameter (wsClient)

.filter(_.isDefined).map(_.get) can be replaced with flatten: mdMetadataSet.linkage.map[Option[info.smart.models.owc100.OwcLink], List[Option[info.smart.models.owc100.OwcLink]]](((x$2: models.gmd.CIOnlineResource) => OwcGeoJsonConverters.this.asOwcLink(x$2)))(immutable.this.List.canBuildFrom[Option[info.smart.models.owc100.OwcLink]]).filter(((x$3: Option[info.smart.models.owc100.OwcLink]) => x$3.isDefined)).map[info.smart.models.owc100.OwcLink, List[info.smart.models.owc100.OwcLink]](((x$4: Option[info.smart.models.owc100.OwcLink]) => x$4.get))

.filter(_.isDefined).map(_.get) can be replaced with flatten: mdMetadataSet.linkage.map[Option[info.smart.models.owc100.OwcLink], List[Option[info.smart.models.owc100.OwcLink]]](((x$5: models.gmd.CIOnlineResource) => OwcGeoJsonConverters.this.asOwcLink(x$5)))(immutable.this.List.canBuildFrom[Option[info.smart.models.owc100.OwcLink]]).filter(((x$6: Option[info.smart.models.owc100.OwcLink]) => x$6.isDefined)).map[info.smart.models.owc100.OwcLink, List[info.smart.models.owc100.OwcLink]](((x$7: Option[info.smart.models.owc100.OwcLink]) => x$7.get))

.filter(_.isDefined).map(_.get) can be replaced with flatten: mdMetadataSet.linkage.map[Option[info.smart.models.owc100.OwcLink], List[Option[info.smart.models.owc100.OwcLink]]](((x$8: models.gmd.CIOnlineResource) => OwcGeoJsonConverters.this.asOwcLink(x$8)))(immutable.this.List.canBuildFrom[Option[info.smart.models.owc100.OwcLink]]).filter(((x$9: Option[info.smart.models.owc100.OwcLink]) => x$9.isDefined)).map[info.smart.models.owc100.OwcLink, List[info.smart.models.owc100.OwcLink]](((x$10: Option[info.smart.models.owc100.OwcLink]) => x$10.get))

.filter(_.isDefined).map(_.get) can be replaced with flatten: mdMetadataSet.linkage.map[Option[info.smart.models.owc100.OwcLink], List[Option[info.smart.models.owc100.OwcLink]]](((x$11: models.gmd.CIOnlineResource) => OwcGeoJsonConverters.this.asOwcLink(x$11)))(immutable.this.List.canBuildFrom[Option[info.smart.models.owc100.OwcLink]]).filter(((x$12: Option[info.smart.models.owc100.OwcLink]) => x$12.isDefined)).map[info.smart.models.owc100.OwcLink, List[info.smart.models.owc100.OwcLink]](((x$13: Option[info.smart.models.owc100.OwcLink]) => x$13.get))

x$4.get

x$7.get

x$10.get

x$13.get

dateRange.head

Methods should be in camelCase style with the first letter lower-case. See http://docs.scala-lang.org/style/naming-conventions.html#methods

Methods should be in camelCase style with the first letter lower-case. See http://docs.scala-lang.org/style/naming-conventions.html#methods

Methods should be in camelCase style with the first letter lower-case. See http://docs.scala-lang.org/style/naming-conventions.html#methods

Methods should be in camelCase style with the first letter lower-case. See http://docs.scala-lang.org/style/naming-conventions.html#methods

Methods should be in camelCase style with the first letter lower-case. See http://docs.scala-lang.org/style/naming-conventions.html#methods

Methods should be in camelCase style with the first letter lower-case. See http://docs.scala-lang.org/style/naming-conventions.html#methods

asInstanceOf used near r.asInstanceOf[(String, String, String)]. Consider using pattern matching.

asInstanceOf used near l.asInstanceOf[List[(String, String, String)]]. Consider using pattern matching.

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.