diff --git a/src/main/scala/codecheck/github/models/Issue.scala b/src/main/scala/codecheck/github/models/Issue.scala index ab9e0f5..8d1e261 100644 --- a/src/main/scala/codecheck/github/models/Issue.scala +++ b/src/main/scala/codecheck/github/models/Issue.scala @@ -8,6 +8,7 @@ import org.json4s.JInt import org.json4s.JArray import org.json4s.JsonDSL._ import org.joda.time.DateTime +import org.joda.time.DateTimeZone import codecheck.github.utils.ToDo @@ -77,7 +78,7 @@ case class IssueListOption( ) { def q = s"?filter=$filter&state=$state&sort=$sort&direction=$direction" + (if (!labels.isEmpty) "&labels=" + labels.mkString(",") else "") + - (if (!since.isEmpty) (since map ("&since=" + _.toString("yyyy-MM-dd'T'HH:mm:ss'Z'"))).get else "") + since.map("&since=" + _.toDateTime(DateTimeZone.UTC).toString("yyyy-MM-dd'T'HH:mm:ss'Z'")).getOrElse("") } case class IssueListOption4Repository( @@ -91,15 +92,13 @@ case class IssueListOption4Repository( direction: SortDirection = SortDirection.desc, since: Option[DateTime] = None ) { - def q = "?" + (if (!milestone.isEmpty) (milestone map (t => s"milestone=$t&")).get else "") + - s"state=$state" + - (if (!assignee.isEmpty) (assignee map (t => s"&assignee=$t")).get else "") + - (if (!creator.isEmpty) (creator map (t => s"&creator=$t")).get else "") + - (if (!mentioned.isEmpty) (mentioned map (t => s"&mentioned=$t")).get else "") + + def q = s"?state=$state&sort=$sort&direction=$direction" + + milestone.map(t => s"milestone=$t&").getOrElse("") + + assignee.map(t => s"&assignee=$t").getOrElse("") + + creator.map(t => s"&creator=$t").getOrElse("") + + mentioned.map(t => s"&mentioned=$t").getOrElse("") + (if (!labels.isEmpty) "&labels=" + labels.mkString(",") else "") + - s"&sort=$sort" + - s"&direction=$direction" + - (if (!since.isEmpty) (since map ("&since=" + _.toString("yyyy-MM-dd'T'HH:mm:ss'Z'"))).get else "") + since.map("&since=" + _.toDateTime(DateTimeZone.UTC).toString("yyyy-MM-dd'T'HH:mm:ss'Z'")).getOrElse("") } case class IssueInput( diff --git a/src/test/scala/IssueOpSpec.scala b/src/test/scala/IssueOpSpec.scala index 533c1c0..6d1485f 100644 --- a/src/test/scala/IssueOpSpec.scala +++ b/src/test/scala/IssueOpSpec.scala @@ -22,7 +22,7 @@ class IssueOpSpec extends FunSpec with Constants with BeforeAndAfterAll { val number = 1 var nUser: Long = 0 var nOrg: Long = 0 - var nTime: DateTime = DateTime.now().toDateTime(DateTimeZone.UTC) + var nTime: DateTime = DateTime.now val tRepo = repo + "2" override def beforeAll() {