diff --git a/src/main/scala/app/AccountController.scala b/src/main/scala/app/AccountController.scala index d8e6362..4586186 100644 --- a/src/main/scala/app/AccountController.scala +++ b/src/main/scala/app/AccountController.scala @@ -76,7 +76,7 @@ updateImage(userName, form.fileId, form.clearImage) flash += "info" -> "Account information has been updated." - redirect("/%s/_edit".format(userName)) + redirect(s"/${userName}/_edit") } getOrElse NotFound }) diff --git a/src/main/scala/app/IndexController.scala b/src/main/scala/app/IndexController.scala index 7eff0cd..c16873b 100644 --- a/src/main/scala/app/IndexController.scala +++ b/src/main/scala/app/IndexController.scala @@ -9,10 +9,13 @@ with SystemSettingsService with ActivityService => get("/"){ + val loginAccount = context.loginAccount + html.index(getRecentActivities(), - getAccessibleRepositories(context.loginAccount, baseUrl), + getAccessibleRepositories(loginAccount, baseUrl), loadSystemSettings(), - context.loginAccount.map{ account => getRepositoryNamesOfUser(account.userName) }.getOrElse(Nil)) + loginAccount.map{ account => getRepositoryNamesOfUser(account.userName) }.getOrElse(Nil) + ) } } \ No newline at end of file diff --git a/src/main/scala/app/LabelsController.scala b/src/main/scala/app/LabelsController.scala index e01c63e..e666d67 100644 --- a/src/main/scala/app/LabelsController.scala +++ b/src/main/scala/app/LabelsController.scala @@ -24,7 +24,7 @@ post("/:owner/:repository/issues/label/new", newForm)(collaboratorsOnly { (form, repository) => createLabel(repository.owner, repository.name, form.labelName, form.color.substring(1)) - redirect("/%s/%s/issues".format(repository.owner, repository.name)) + redirect(s"/${repository.owner}/${repository.name}/issues") }) ajaxGet("/:owner/:repository/issues/label/edit")(collaboratorsOnly { repository => @@ -53,9 +53,9 @@ private def labelName: Constraint = new Constraint(){ def validate(name: String, value: String): Option[String] = if(!value.matches("^[^,]+$")){ - Some("%s contains invalid character.".format(name)) + Some(s"${name} contains invalid character.") } else if(value.startsWith("_") || value.startsWith("-")){ - Some("%s starts with invalid character.".format(name)) + Some(s"${name} starts with invalid character.") } else { None } diff --git a/src/main/scala/app/MilestonesController.scala b/src/main/scala/app/MilestonesController.scala index 480b8bc..55c60d0 100644 --- a/src/main/scala/app/MilestonesController.scala +++ b/src/main/scala/app/MilestonesController.scala @@ -35,7 +35,7 @@ post("/:owner/:repository/issues/milestones/new", milestoneForm)(collaboratorsOnly { (form, repository) => createMilestone(repository.owner, repository.name, form.title, form.description, form.dueDate) - redirect("/%s/%s/issues/milestones".format(repository.owner, repository.name)) + redirect(s"/${repository.owner}/${repository.name}/issues/milestones") }) get("/:owner/:repository/issues/milestones/:milestoneId/edit")(collaboratorsOnly { repository => @@ -45,28 +45,28 @@ post("/:owner/:repository/issues/milestones/:milestoneId/edit", milestoneForm)(collaboratorsOnly { (form, repository) => getMilestone(repository.owner, repository.name, params("milestoneId").toInt).map { milestone => updateMilestone(milestone.copy(title = form.title, description = form.description, dueDate = form.dueDate)) - redirect("/%s/%s/issues/milestones".format(repository.owner, repository.name)) + redirect(s"/${repository.owner}/${repository.name}/issues/milestones") } getOrElse NotFound }) get("/:owner/:repository/issues/milestones/:milestoneId/close")(collaboratorsOnly { repository => getMilestone(repository.owner, repository.name, params("milestoneId").toInt).map { milestone => closeMilestone(milestone) - redirect("/%s/%s/issues/milestones".format(repository.owner, repository.name)) + redirect(s"/${repository.owner}/${repository.name}/issues/milestones") } getOrElse NotFound }) get("/:owner/:repository/issues/milestones/:milestoneId/open")(collaboratorsOnly { repository => getMilestone(repository.owner, repository.name, params("milestoneId").toInt).map { milestone => openMilestone(milestone) - redirect("/%s/%s/issues/milestones".format(repository.owner, repository.name)) + redirect(s"/${repository.owner}/${repository.name}/issues/milestones") } getOrElse NotFound }) get("/:owner/:repository/issues/milestones/:milestoneId/delete")(collaboratorsOnly { repository => getMilestone(repository.owner, repository.name, params("milestoneId").toInt).map { milestone => deleteMilestone(repository.owner, repository.name, milestone.milestoneId) - redirect("/%s/%s/issues/milestones".format(repository.owner, repository.name)) + redirect(s"/${repository.owner}/${repository.name}/issues/milestones") } getOrElse NotFound }) diff --git a/src/main/scala/app/RepositorySettingsController.scala b/src/main/scala/app/RepositorySettingsController.scala index a5c79b4..08f6474 100644 --- a/src/main/scala/app/RepositorySettingsController.scala +++ b/src/main/scala/app/RepositorySettingsController.scala @@ -31,7 +31,7 @@ * Redirect to the Options page. */ get("/:owner/:repository/settings")(ownerOnly { repository => - redirect("/%s/%s/settings/options".format(repository.owner, repository.name)) + redirect(s"/${repository.owner}/${repository.name}/settings/options") }) /** @@ -47,7 +47,7 @@ post("/:owner/:repository/settings/options", optionsForm)(ownerOnly { (form, repository) => saveRepositoryOptions(repository.owner, repository.name, form.description, form.defaultBranch, form.isPrivate) flash += "info" -> "Repository settings has been updated." - redirect("/%s/%s/settings/options".format(repository.owner, repository.name)) + redirect(s"/${repository.owner}/${repository.name}/settings/options") }) /** @@ -70,7 +70,7 @@ */ post("/:owner/:repository/settings/collaborators/add", collaboratorForm)(ownerOnly { (form, repository) => addCollaborator(repository.owner, repository.name, form.userName) - redirect("/%s/%s/settings/collaborators".format(repository.owner, repository.name)) + redirect(s"/${repository.owner}/${repository.name}/settings/collaborators") }) /** @@ -78,7 +78,7 @@ */ get("/:owner/:repository/settings/collaborators/remove")(ownerOnly { repository => removeCollaborator(repository.owner, repository.name, params("name")) - redirect("/%s/%s/settings/collaborators".format(repository.owner, repository.name)) + redirect(s"/${repository.owner}/${repository.name}/settings/collaborators") }) /** @@ -98,7 +98,7 @@ FileUtils.deleteDirectory(getWikiRepositoryDir(repository.owner, repository.name)) FileUtils.deleteDirectory(getTemporaryDir(repository.owner, repository.name)) - redirect("/%s".format(repository.owner)) + redirect(s"/${repository.owner}") }) /** diff --git a/src/main/scala/app/WikiController.scala b/src/main/scala/app/WikiController.scala index 405733d..8a4a22f 100644 --- a/src/main/scala/app/WikiController.scala +++ b/src/main/scala/app/WikiController.scala @@ -32,7 +32,7 @@ get("/:owner/:repository/wiki")(referrersOnly { repository => getWikiPage(repository.owner, repository.name, "Home").map { page => wiki.html.page("Home", page, repository, hasWritePermission(repository.owner, repository.name, context.loginAccount)) - } getOrElse redirect("/%s/%s/wiki/Home/_edit".format(repository.owner, repository.name)) + } getOrElse redirect(s"/${repository.owner}/${repository.name}/wiki/Home/_edit") }) get("/:owner/:repository/wiki/:page")(referrersOnly { repository => @@ -40,7 +40,7 @@ getWikiPage(repository.owner, repository.name, pageName).map { page => wiki.html.page(pageName, page, repository, hasWritePermission(repository.owner, repository.name, context.loginAccount)) - } getOrElse redirect("/%s/%s/wiki/%s/_edit".format(repository.owner, repository.name, pageName)) // TODO URLEncode + } getOrElse redirect(s"/${repository.owner}/${repository.name}/wiki/${pageName}/_edit") // TODO URLEncode }) get("/:owner/:repository/wiki/:page/_history")(referrersOnly { repository => @@ -82,7 +82,7 @@ updateLastActivityDate(repository.owner, repository.name) recordEditWikiPageActivity(repository.owner, repository.name, loginAccount.userName, form.pageName) - redirect("/%s/%s/wiki/%s".format(repository.owner, repository.name, form.pageName)) + redirect(s"/${repository.owner}/${repository.name}/wiki/${form.pageName}") }) get("/:owner/:repository/wiki/_new")(collaboratorsOnly { @@ -98,16 +98,16 @@ updateLastActivityDate(repository.owner, repository.name) recordCreateWikiPageActivity(repository.owner, repository.name, loginAccount.userName, form.pageName) - redirect("/%s/%s/wiki/%s".format(repository.owner, repository.name, form.pageName)) + redirect(s"/${repository.owner}/${repository.name}/wiki/${form.pageName}") }) get("/:owner/:repository/wiki/:page/_delete")(collaboratorsOnly { repository => val pageName = params("page") - deleteWikiPage(repository.owner, repository.name, pageName, context.loginAccount.get.userName, "Delete %s".format(pageName)) + deleteWikiPage(repository.owner, repository.name, pageName, context.loginAccount.get.userName, s"Delete ${pageName}") updateLastActivityDate(repository.owner, repository.name) - redirect("/%s/%s/wiki".format(repository.owner, repository.name)) + redirect(s"/${repository.owner}/${repository.name}/wiki") }) get("/:owner/:repository/wiki/_pages")(referrersOnly { repository => diff --git a/src/main/scala/servlet/AutoUpdateListener.scala b/src/main/scala/servlet/AutoUpdateListener.scala index 266e617..dbfe8cb 100644 --- a/src/main/scala/servlet/AutoUpdateListener.scala +++ b/src/main/scala/servlet/AutoUpdateListener.scala @@ -25,7 +25,7 @@ * If corresponding SQL file does not exist, this method do nothing. */ def update(conn: Connection): Unit = { - val sqlPath = "update/%d_%d.sql".format(majorVersion, minorVersion) + val sqlPath = s"update/${majorVersion}_${minorVersion}.sql" val in = Thread.currentThread.getContextClassLoader.getResourceAsStream(sqlPath) if(in != null){ val sql = IOUtils.toString(in, "UTF-8") @@ -42,7 +42,7 @@ /** * MAJOR.MINOR */ - val versionString = "%d.%d".format(majorVersion, minorVersion) + val versionString = s"${majorVersion}.${minorVersion}" } /** diff --git a/src/main/scala/util/Directory.scala b/src/main/scala/util/Directory.scala index b378a99..f2b7f4c 100644 --- a/src/main/scala/util/Directory.scala +++ b/src/main/scala/util/Directory.scala @@ -13,13 +13,13 @@ val GitBucketConf = new File(GitBucketHome, "gitbucket.conf") - val RepositoryHome = "%s/repositories".format(GitBucketHome) + val RepositoryHome = s"${GitBucketHome}/repositories" /** * Repository names of the specified user. */ def getRepositories(owner: String): List[String] = { - val dir = new File("%s/%s".format(RepositoryHome, owner)) + val dir = new File(s"${RepositoryHome}/${owner}") if(dir.exists){ dir.listFiles.filter { file => file.isDirectory && !file.getName.endsWith(".wiki.git") @@ -33,24 +33,24 @@ * Substance directory of the repository. */ def getRepositoryDir(owner: String, repository: String): File = - new File("%s/%s/%s.git".format(RepositoryHome, owner, repository)) + new File(s"${RepositoryHome}/${owner}/${repository}.git") /** * Directory for uploaded files by the specified user. */ - def getUserUploadDir(userName: String): File = new File("%s/data/%s/files".format(GitBucketHome, userName)) + def getUserUploadDir(userName: String): File = new File(s"${GitBucketHome}/data/${userName}/files") /** * Root of temporary directories for the specified repository. */ def getTemporaryDir(owner: String, repository: String): File = - new File("%s/tmp/%s/%s".format(GitBucketHome, owner, repository)) + new File(s"${GitBucketHome}/tmp/${owner}/${repository}") /** * Temporary directory which is used to create an archive to download repository contents. */ def getDownloadWorkDir(owner: String, repository: String, sessionId: String): File = - new File(getTemporaryDir(owner, repository), "download/%s".format(sessionId)) + new File(getTemporaryDir(owner, repository), s"download/${sessionId}") /** * Temporary directory which is used in the repository creation. @@ -65,7 +65,7 @@ * Substance directory of the wiki repository. */ def getWikiRepositoryDir(owner: String, repository: String): File = - new File("%s/%s/%s.wiki.git".format(Directory.RepositoryHome, owner, repository)) + new File(s"${RepositoryHome}/${owner}/${repository}.wiki.git") /** * Wiki working directory which is cloned from the wiki repository. diff --git a/src/main/scala/util/FileUploadUtil.scala b/src/main/scala/util/FileUploadUtil.scala index 61f2718..6efacc1 100644 --- a/src/main/scala/util/FileUploadUtil.scala +++ b/src/main/scala/util/FileUploadUtil.scala @@ -11,7 +11,7 @@ new SimpleDateFormat("yyyyMMddHHmmSSsss").format(new java.util.Date(System.currentTimeMillis)) def TemporaryDir(implicit session: HttpSession): java.io.File = - new java.io.File(GitBucketHome, "tmp/_upload/%s".format(session.getId)) + new java.io.File(GitBucketHome, s"tmp/_upload/${session.getId}") def getTemporaryFile(fileId: String)(implicit session: HttpSession): java.io.File = new java.io.File(TemporaryDir, fileId) diff --git a/src/main/scala/util/JGitUtil.scala b/src/main/scala/util/JGitUtil.scala index 9bde1e8..7866799 100644 --- a/src/main/scala/util/JGitUtil.scala +++ b/src/main/scala/util/JGitUtil.scala @@ -153,7 +153,7 @@ } RepositoryInfo( - owner, repository, baseUrl + "/git/%s/%s.git".format(owner, repository), + owner, repository, s"${baseUrl}/git/${owner}/${repository}.git", // commit count commitCount, // branches @@ -169,7 +169,7 @@ } catch { // not initialized case e: NoHeadException => RepositoryInfo( - owner, repository, baseUrl + "/git/%s/%s.git".format(owner, repository), 0, Nil, Nil) + owner, repository, s"${baseUrl}/git/${owner}/${repository}.git", 0, Nil, Nil) } } diff --git a/src/main/scala/util/Validations.scala b/src/main/scala/util/Validations.scala index a518978..1d42d99 100644 --- a/src/main/scala/util/Validations.scala +++ b/src/main/scala/util/Validations.scala @@ -11,9 +11,9 @@ def identifier: Constraint = new Constraint(){ def validate(name: String, value: String): Option[String] = if(!value.matches("^[a-zA-Z0-9\\-_]+$")){ - Some("%s contains invalid character.".format(name)) + Some(s"${name} contains invalid character.") } else if(value.startsWith("_") || value.startsWith("-")){ - Some("%s starts with invalid character.".format(name)) + Some(s"${name} starts with invalid character.") } else { None } diff --git a/src/main/scala/view/Markdown.scala b/src/main/scala/view/Markdown.scala index c8eb148..91c4945 100644 --- a/src/main/scala/view/Markdown.scala +++ b/src/main/scala/view/Markdown.scala @@ -101,7 +101,7 @@ override def visit(node: TextNode) { // convert commit id to link. val text = if(enableCommitLink) node.getText.replaceAll("(^|\\W)([0-9a-f]{40})(\\W|$)", - "$2".format(context.path, repository.owner, repository.name)) + s"""$$2""") else node.getText if (abbreviations.isEmpty) { @@ -116,7 +116,7 @@ if(enableIssueLink && text.matches("#[\\d]+")){ // convert issue id to link val issueId = text.substring(1).toInt - printer.print("#%d".format(context.path, repository.owner, repository.name, issueId, issueId)) + printer.print(s"""#${issueId}""") } else { printTag(node, "h" + node.getLevel) } diff --git a/src/main/scala/view/helpers.scala b/src/main/scala/view/helpers.scala index b12f644..d80a4d8 100644 --- a/src/main/scala/view/helpers.scala +++ b/src/main/scala/view/helpers.scala @@ -35,9 +35,7 @@ Html(Markdown.toHtml(value, repository, enableWikiLink, enableCommitLink, enableIssueLink)) } - def activityMessage(message: String)(implicit context: app.Context): Html = { - val a = s"a $message aa $$1 a" - + def activityMessage(message: String)(implicit context: app.Context): Html = Html(message .replaceAll("\\[issue:([^\\s]+?)/([^\\s]+?)#((\\d+))\\]" , s"""$$1/$$2#$$3""") .replaceAll("\\[repo:([^\\s]+?)/([^\\s]+?)\\]" , s"""$$1/$$2""") @@ -45,23 +43,24 @@ .replaceAll("\\[tag:([^\\s]+?)/([^\\s]+?)#([^\\s]+?)\\]" , s"""$$3""") .replaceAll("\\[user:([^\\s]+?)\\]" , s"""$$1""") ) - } /** * Generates the url to the repository. */ def url(repository: service.RepositoryService.RepositoryInfo)(implicit context: app.Context): String = - "%s/%s/%s".format(context.path, repository.owner, repository.name) + s"${context.path}/${repository.owner}/${repository.name}" /** * Generates the url to the account page. */ - def url(userName: String)(implicit context: app.Context): String = "%s/%s".format(context.path, userName) + def url(userName: String)(implicit context: app.Context): String = + s"${context.path}/${userName}" /** * Returns the url to the root of assets. */ - def assets(implicit context: app.Context): String = "%s/assets".format(context.path) + def assets(implicit context: app.Context): String = + s"${context.path}/assets" /** * Converts issue id and commit id to link. @@ -71,9 +70,9 @@ // escape HTML tags .replaceAll("&", "&").replaceAll("<", "<").replaceAll(">", ">").replaceAll("\"", """) // convert issue id to link - .replaceAll("(^|\\W)#(\\d+)(\\W|$)", "$1#$2$3".format(context.path, repository.owner, repository.name)) + .replaceAll("(^|\\W)#(\\d+)(\\W|$)", s"""$$1#$$2$$3""") // convert commit id to link - .replaceAll("(^|\\W)([a-f0-9]{40})(\\W|$)", "$1$2$3").format(context.path, repository.owner, repository.name)) + .replaceAll("(^|\\W)([a-f0-9]{40})(\\W|$)", s"""$$1$$2$$3""")) /**