Yi EungJun 2014-03-04
Fix java accessor method called as empty-paren
Methods that follow JavaBean naming contract for accessor (e.g. getXXX,
setXXX, isXXX, ..) are expected to have no side effects. So you don't
need to call them as empty-paren.
@33ab80d35d57ffa64b3d7fa5d4c3e3613c9a92f9
app/views/board/list.scala.html
--- app/views/board/list.scala.html
+++ app/views/board/list.scala.html
@@ -64,7 +64,7 @@
     	}
 
     	<ul class="post-list-wrap">
-    	@for(post <- page.getList()){
+	@for(post <- page.getList){
             @partial_list(post, project)
         }
     	</ul>
app/views/board/view.scala.html
--- app/views/board/view.scala.html
+++ app/views/board/view.scala.html
@@ -153,7 +153,7 @@
         // yobi.Mention
         yobi.Mention({
             "target": "comment-editor",
-            "url"   : "@Html(routes.ProjectApp.mentionList(project.owner, project.name, post.getNumber, post.asResource().getType().resource()).toString())"
+            "url"   : "@Html(routes.ProjectApp.mentionList(project.owner, project.name, post.getNumber, post.asResource().getType.resource()).toString())"
         });
 	});
 </script>
app/views/code/branches.scala.html
--- app/views/code/branches.scala.html
+++ app/views/code/branches.scala.html
@@ -35,7 +35,7 @@
                     <li>
                         <a href="@routes.CodeHistoryApp.historyUntilHead(project.owner, project.name)">@Messages("code.commits")</a>
                     </li>
-                    @if(project.isGit()) {
+                    @if(project.isGit) {
                     <li class="active">
                         <a href="@routes.BranchApp.branches(project.owner, project.name)">@Messages("title.branches")</a>
                     </li>
app/views/code/partial_branchrow.scala.html
--- app/views/code/partial_branchrow.scala.html
+++ app/views/code/partial_branchrow.scala.html
@@ -58,7 +58,7 @@
     </td>
     <td class="commitMessage">
         @getCommiterAvatar(branch)
-        @common.commitMsg(branch.getHeadCommit.getShortMessage(),
+        @common.commitMsg(branch.getHeadCommit.getShortMessage,
                           branch.getHeadCommit.getMessage,
                           getShowCommitURL(branch))
     </td>
@@ -68,7 +68,7 @@
     <td class="pullRequest">
     @if(branch.getPullRequest != null) {
         <a href="@routes.PullRequestApp.pullRequest(branch.getPullRequest.toProject.owner, branch.getPullRequest.toProject.name, branch.getPullRequest.number)"
-           class="blue-txt pullrequest-state @branch.getPullRequest.state.toString().toLowerCase()">
+           class="blue-txt pullrequest-state @branch.getPullRequest.state.toString().toLowerCase">
            pullRequest-@branch.getPullRequest.number
         </a>
     } else {
@@ -80,11 +80,11 @@
         isAllowed(UserApp.currentUser(), project.asResource(), Operation.UPDATE)) {
     <td class="actions">
         @if(isAllowed(UserApp.currentUser(), project.asResource(), Operation.UPDATE)){
-            @if(branch.getName() != project.defaultBranch()){
+            @if(branch.getName != project.defaultBranch()){
                 <button type="button"
                         class="ybtn ybtn-default"
                         data-request-method="post"
-                        data-request-uri="@routes.BranchApp.setAsDefault(project.owner, project.name, URLEncoder.encode(branch.getName(), "UTF-8"))">
+                        data-request-uri="@routes.BranchApp.setAsDefault(project.owner, project.name, URLEncoder.encode(branch.getName, "UTF-8"))">
                     @Messages("code.branches.setAsDefault")
                 </button>
             }
app/views/code/view.scala.html
--- app/views/code/view.scala.html
+++ app/views/code/view.scala.html
@@ -46,7 +46,7 @@
                 } else {
                 <li><a href="@routes.CodeHistoryApp.history(project.owner, project.name, URLEncoder.encode(branch, "UTF-8"), null)">@Messages("code.commits")</a></li>
                 }
-                @if(project.isGit()) {
+                @if(project.isGit) {
                 <li>
                     <a href="@routes.BranchApp.branches(project.owner, project.name)">@Messages("title.branches")</a>
                 </li>
app/views/git/partial_diff.scala.html
--- app/views/git/partial_diff.scala.html
+++ app/views/git/partial_diff.scala.html
@@ -51,11 +51,11 @@
 
 <div id="diff">
 @if(result != null) {
-    @if(result.getGitCommits().isEmpty()) {
+    @if(result.getGitCommits().isEmpty) {
         <h4>@Messages("pullRequest.diff.noChanges")</h4>
     } else {
         <div style="margin-top:20px;">
-            @if(result.getGitConflicts() == null){
+            @if(result.getGitConflicts == null){
             <div class="alert alert-success">
                 <h5>@Messages("pullRequest.is.safe")</h5>
             </div>
@@ -82,11 +82,11 @@
                         </tr>
                         </thead>
                         <tbody class="tbody">
-                        @for(commit <- result.getGitCommits()) {
+                        @for(commit <- result.getGitCommits) {
                         <tr>
                             <td class="commit-id">
-                                <a href="@routes.CodeHistoryApp.show(pullRequest.fromProject.owner, pullRequest.fromProject.name, commit.getId())">
-                                    @commit.getShortId()
+                                <a href="@routes.CodeHistoryApp.show(pullRequest.fromProject.owner, pullRequest.fromProject.name, commit.getId)">
+                                    @commit.getShortId
                                     <i class="yobicon-right"></i>
                                 </a>
                             </td>
@@ -96,14 +96,14 @@
                                     <span class="number-of-comments"><i class="yobicon-comments"></i> @numOfComment</span>
                                     }
                                 }
-                                @defining(commit.getMessage()){ commitMsg =>
+                                @defining(commit.getMessage){ commitMsg =>
                                     @common.commitMsg(commitMsg.split("\n")(0),
                                         commitMsg,
-                                        routes.CodeHistoryApp.show(pullRequest.fromProject.owner, pullRequest.fromProject.name, commit.getId()).toString())
+                                        routes.CodeHistoryApp.show(pullRequest.fromProject.owner, pullRequest.fromProject.name, commit.getId).toString())
                                 }
                             </td>
                             <td class="date" title="@JodaDateUtil.getDateString(commit.getAuthorDate)">
-                                @agoString(JodaDateUtil.ago(commit.getAuthorDate()))
+                                @agoString(JodaDateUtil.ago(commit.getAuthorDate))
                             </td>
                             <td class="author @commit.getAuthorEmail">
                                 @defining(User.find.where.eq("email", commit.getAuthorEmail).findUnique) { user =>
@@ -113,7 +113,7 @@
                                 </a>
                                 } else {
                                 <div class="avatar-wrap">
-                                    <img src="@urlToPicture(commit.getAuthorEmail(), 32)" width="32" height="32"/>
+                                    <img src="@urlToPicture(commit.getAuthorEmail, 32)" width="32" height="32"/>
                                 </div>
                                 }
                                 }
app/views/git/partial_forklist.scala.html
--- app/views/git/partial_forklist.scala.html
+++ app/views/git/partial_forklist.scala.html
@@ -2,7 +2,7 @@
 @import utils.TemplateHelper._
 
 
-@if(project.hasForks()) {
+@if(project.hasForks) {
     <div id="forks" class="tab-pane">
         <ul class="all-projects">
             @for(fork <- project.forkingProjects) {
@@ -14,7 +14,7 @@
                         </a>&nbsp;
                         @if(!fork.isPublic){ <i class="yobicon-lock yobicon-small"></i> }
                         @for(label <- fork.labels) {
-                        <span class="project-label @label.category.toLowerCase()">@label.name</span>
+                        <span class="project-label @label.category.toLowerCase">@label.name</span>
                         }
                     </div>
                     <div style="float:left">
@@ -22,7 +22,7 @@
                             <a href="@routes.ProjectApp.project(fork.owner, fork.name)" class="black">@fork.name</a>
                             @if(!fork.isPublic){ <i class="yobicon-lock yobicon-small"></i> }
                             @for(label <- fork.labels) {
-                            <span class="project-label @label.category.toLowerCase()">@label.name</span>
+                            <span class="project-label @label.category.toLowerCase">@label.name</span>
                             }
                         </div>
                         <div class="desc">
app/views/git/partial_list.scala.html
--- app/views/git/partial_list.scala.html
+++ app/views/git/partial_list.scala.html
@@ -7,8 +7,8 @@
     branch.replace("refs/heads/", "")
 }
 <ul class="post-list-wrap">
-@if(page.getList().size()> 0){
-    @for(req <- page.getList().iterator()) {
+@if(page.getList.size()> 0){
+    @for(req <- page.getList.iterator()) {
     @defining(User.findByLoginId(req.contributor.loginId)){ user =>
         <li class="post-item">
             <div class="span10">
@@ -74,6 +74,6 @@
     $(function(){
         $('[data-toggle="tooltip"]').tooltip();
         var welPagination = $("#pagination");
-        yobi.Pagination.update(welPagination, @page.getTotalPageCount());
+        yobi.Pagination.update(welPagination, @page.getTotalPageCount);
     });
 </script>
app/views/git/partial_pull_request_comment.scala.html
--- app/views/git/partial_pull_request_comment.scala.html
+++ app/views/git/partial_pull_request_comment.scala.html
@@ -52,7 +52,7 @@
                     <strong>@comment.authorLoginId</strong>
                 </a>
             </span>
-            <a href="#comment-@comment.id" class="ago">@momentFromNow(comment.getDate(), lang.language)</a>
+            <a href="#comment-@comment.id" class="ago">@momentFromNow(comment.getDate, lang.language)</a>
 
             @if(isLineComment(comment) && comment.isOutdated) {
             <span class="outdated-message" style="color: red">@Messages("code.outdatedDiff")</span>
app/views/git/partial_state.scala.html
--- app/views/git/partial_state.scala.html
+++ app/views/git/partial_state.scala.html
@@ -88,7 +88,7 @@
     </div>
 }
 
-@if(pull.isMerged()){
+@if(pull.isMerged){
 <div class="alert alert-info">
     <a href="@routes.UserApp.userInfo(pull.receiver.loginId)" class="avatar-wrap small">
         <img src="@pull.receiver.avatarUrl" width="25" height="25">
app/views/git/view.scala.html
--- app/views/git/view.scala.html
+++ app/views/git/view.scala.html
@@ -51,7 +51,7 @@
                     }
                 </button>
                 }
-                <div id="actOnOpen" class="mr5 pull-right" style="@if(pull.isMerging == false && pull.isOpen()){display:block;}else{display:none;}">
+                <div id="actOnOpen" class="mr5 pull-right" style="@if(pull.isMerging == false && pull.isOpen){display:block;}else{display:none;}">
                     @if(project.isUsingReviewerCount && isAllowed(UserApp.currentUser(), pull.asResource(), Operation.ACCEPT)) {
                         @if(pull.isReviewedBy(UserApp.currentUser())) {
                             <a data-request-method="post" class="ybtn" href="@routes.ReviewApp.unreview(project.owner, project.name, pull.number)">
@@ -67,7 +67,7 @@
                         @if(pull.isReviewed || !project.isUsingReviewerCount) {
                             <a data-request-method="post" id="btnAccept" href="@routes.PullRequestApp.accept(project.owner, project.name, pull.number)" class="ybtn ybtn-success" @if(pull.isConflict == true){style="display:none;"}>@Messages("pullRequest.accept")</a>
                         } else {
-                            <button class="ybtn ybtn-disabled" data-toggle="tooltip" data-placement="top" title="@Messages("pullRequest.not.enough.review.point", pull.getLackingReviewerCount())">@Messages("pullRequest.accept")</button>
+                            <button class="ybtn ybtn-disabled" data-toggle="tooltip" data-placement="top" title="@Messages("pullRequest.not.enough.review.point", pull.getLackingReviewerCount)">@Messages("pullRequest.accept")</button>
                         }
 
                     }
@@ -79,7 +79,7 @@
                     }
                 </div>
 
-                <div id="actOnClosed" class="mr5 pull-right" style="@if(pull.isMerging == false && pull.isClosed()) {display:block;}else{display:none;}">
+                <div id="actOnClosed" class="mr5 pull-right" style="@if(pull.isMerging == false && pull.isClosed) {display:block;}else{display:none;}">
                     @if(isAllowed(UserApp.currentUser(), pull.asResource(), Operation.REOPEN)) {
                         <a data-request-method="post" href="@routes.PullRequestApp.open(project.owner, project.name, pull.number)" class="ybtn">@Messages("pullRequest.reopen")</a>
                     }
@@ -121,7 +121,7 @@
 
         yobi.Mention({
             target:'comment-editor',
-            @if(pull.getCurrentCommits.isEmpty()) {
+            @if(pull.getCurrentCommits.isEmpty) {
                 url : "@Html(routes.ProjectApp.mentionListAtPullRequest(pull.toProject.owner, pull.toProject.name, "", pull.id).toString())"
             } else {
                 url : "@Html(routes.ProjectApp.mentionListAtPullRequest(pull.toProject.owner, pull.toProject.name, pull.getCurrentCommits.get(0).getCommitId, pull.id).toString())"
app/views/git/viewChanges.scala.html
--- app/views/git/viewChanges.scala.html
+++ app/views/git/viewChanges.scala.html
@@ -50,7 +50,7 @@
     $(document).ready(function() {
         yobi.Mention({
             target:'comment-editor',
-            @if(pull.getCurrentCommits.isEmpty()) {
+            @if(pull.getCurrentCommits.isEmpty) {
                 url : "@Html(routes.ProjectApp.mentionListAtPullRequest(pull.toProject.owner, pull.toProject.name, "", pull.id).toString())"
             } else {
                 url : "@Html(routes.ProjectApp.mentionListAtPullRequest(pull.toProject.owner, pull.toProject.name, pull.getCurrentCommits.get(0).getCommitId, pull.id).toString())"
app/views/git/viewCommits.scala.html
--- app/views/git/viewCommits.scala.html
+++ app/views/git/viewCommits.scala.html
@@ -33,8 +33,8 @@
                     @for(commit <- pull.getCurrentCommits) {
                     <tr>
                         <td class="commit-id">
-                            <a href="@routes.PullRequestApp.commitView(pull.toProject.owner, pull.toProject.name, pull.number, commit.getCommitId())">
-                                @commit.getCommitShortId()
+                            <a href="@routes.PullRequestApp.commitView(pull.toProject.owner, pull.toProject.name, pull.number, commit.getCommitId)">
+                                @commit.getCommitShortId
                             </a>
                         </td>
 
@@ -46,14 +46,14 @@
                                 }
                             }
 
-                            @defining(commit.getCommitMessage()){ commitMsg =>
+                            @defining(commit.getCommitMessage){ commitMsg =>
                                 @common.commitMsg(commitMsg.split("\n")(0),
                                     commitMsg,
-                                    routes.PullRequestApp.commitView(pull.toProject.owner, pull.toProject.name, pull.number, commit.getCommitId()).toString())
+                                    routes.PullRequestApp.commitView(pull.toProject.owner, pull.toProject.name, pull.number, commit.getCommitId).toString())
                             }
                         </td>
                         <td class="date" title="@JodaDateUtil.getDateString(commit.getAuthorDate)">
-                            @agoString(JodaDateUtil.ago(commit.getAuthorDate()))
+                            @agoString(JodaDateUtil.ago(commit.getAuthorDate))
                         </td>
                         <td class="author @commit.getAuthorEmail">
                             @defining(User.find.where.eq("email", commit.getAuthorEmail).findUnique) { user =>
@@ -63,7 +63,7 @@
                             </a>
                             } else {
                             <div class="avatar-wrap">
-                                <img src="@urlToPicture(commit.getAuthorEmail(), 32)" width="32" height="32"/>
+                                <img src="@urlToPicture(commit.getAuthorEmail, 32)" width="32" height="32"/>
                             </div>
                             }
                             }
app/views/issue/my_partial_search.scala.html
--- app/views/issue/my_partial_search.scala.html
+++ app/views/issue/my_partial_search.scala.html
@@ -77,7 +77,7 @@
             @for(state <- Array(State.OPEN, State.CLOSED)) {
                 <li @if(param.state == state.state) { class="active" } data-pjax>
                     <a href="#" state="@state.state">
-                        @Messages("issue.state." + state.name.toLowerCase())
+                        @Messages("issue.state." + state.name.toLowerCase)
                         <span class="num-badge">@Issue.countIssuesBy(param.clone.setState(state))</span>
                     </a>
                 </li>
@@ -85,9 +85,9 @@
             </ul>
 
 
-        @if(currentPage.getList().size() > 0){
+        @if(currentPage.getList.size() > 0){
 		<div class="filter-wrap small-heights">
-                @if(currentPage.getList().size() > 1){
+                @if(currentPage.getList.size() > 1){
 			<div class="filters pull-right">
 				@makeFilterLink("updatedDate", param.orderBy, param.orderDir, Messages("common.order.updatedDate"))
 				@makeFilterLink("createdDate", param.orderBy, param.orderDir, Messages("common.order.date"))
@@ -124,7 +124,7 @@
                 "welSearchForm":$("form[name='search']"),
                 "welFilter": $("a[pjax-filter]"),
                 "elPagination": $("#pagination"),
-                "nTotalPages" : @currentPage.getTotalPageCount()
+                "nTotalPages" : @currentPage.getTotalPageCount
 				});
 			});
 		</script>
app/views/issue/partial_search.scala.html
--- app/views/issue/partial_search.scala.html
+++ app/views/issue/partial_search.scala.html
@@ -38,7 +38,7 @@
                         <span class="num-badge pull-right">@Issue.countIssuesBy(project.id, paramForEveryone)</span>
                     </a>
                 </li>
-                @if(!UserApp.currentUser().isSiteManager()){
+                @if(!UserApp.currentUser().isSiteManager){
                 <li @if(param.assigneeId == UserApp.currentUser().id){ class="active"}>
                     <a pjax-filter href="#" authorId="" assigneeId="@User.findByLoginId(session.get("loginId")).id" milestoneId="@param.milestoneId">@Messages("issue.list.assignedToMe")
                         <span class="num-badge pull-right">@Issue.countIssuesBy(project.id, paramForIssuesAssignedToMe)</span>
@@ -167,20 +167,20 @@
             @for(state <- Array(State.OPEN, State.CLOSED)) {
                 <li @if(param.state == state.state) { class="active" } data-pjax>
                     <a href="#" state="@state.state">
-                        @Messages("issue.state." + state.name.toLowerCase())
+                        @Messages("issue.state." + state.name.toLowerCase)
                         <span class="num-badge">@Issue.countIssuesBy(project.id, param.clone.setState(state))</span>
                     </a>
                 </li>
             }
             </ul>
 
-        @if(currentPage.getList().size() > 0){
+        @if(currentPage.getList.size() > 0){
             <div class="filter-wrap board">
                 @if(ProjectUser.isMember(UserApp.currentUser().id, project.id)){
                     @partial_massupdate(project, param)
                 }
 
-                @if(currentPage.getList().size() > 1){
+                @if(currentPage.getList.size() > 1){
                 <div class="filters pull-right">
                     @makeFilterLink("updatedDate", param.orderBy, param.orderDir, Messages("common.order.updatedDate"))
                     @makeFilterLink("createdDate", param.orderBy, param.orderDir, Messages("common.order.date"))
@@ -223,7 +223,7 @@
                     "welSearchForm" : $("form[name='search']"),
                     "welFilter"     : $("a[pjax-filter]"),
                     "elPagination"  : $("#pagination"),
-                    "nTotalPages"   : @currentPage.getTotalPageCount()
+                    "nTotalPages" : @currentPage.getTotalPageCount
                 });
 
                 // ShortcutKey
app/views/issue/view.scala.html
--- app/views/issue/view.scala.html
+++ app/views/issue/view.scala.html
@@ -322,7 +322,7 @@
 
         yobi.Mention({
             target:'comment-editor',
-            url : "@Html(routes.ProjectApp.mentionList(project.owner, project.name, issue.getNumber, issue.asResource().getType().resource()).toString())"
+            url : "@Html(routes.ProjectApp.mentionList(project.owner, project.name, issue.getNumber, issue.asResource().getType.resource()).toString())"
         });
     });
 </script>
app/views/milestone/list.scala.html
--- app/views/milestone/list.scala.html
+++ app/views/milestone/list.scala.html
@@ -51,7 +51,7 @@
             </ul>
         </div>
 
-    	@if(milestones == null || milestones.isEmpty()){
+	@if(milestones == null || milestones.isEmpty){
 
     	<div class="error-wrap">
     		<i class="ico ico-err1"></i>
@@ -85,7 +85,7 @@
                         @if(milestone.state != State.CLOSED) {
                             @if(milestone.dueDate != null) {
                             <span class="sp">|</span>
-                            <span class="due-date @if(milestone.isOverDueDate()){over}">
+                            <span class="due-date @if(milestone.isOverDueDate){over}">
                                 @Messages("label.dueDate")
                                 <strong>@milestone.getDueDateString</strong>
                                 <span class="date">
app/views/project/header.scala.html
--- app/views/project/header.scala.html
+++ app/views/project/header.scala.html
@@ -13,7 +13,7 @@
             <div class="project-header-avatar">
                 <img src="@urlToProjectLogo(project)" />
             </div>
-            <div class="project-breadcrumb-wrap @if(project.isForkedFromOrigin()){fork}">
+            <div class="project-breadcrumb-wrap @if(project.isForkedFromOrigin){fork}">
                 <div class="project-breadcrumb">
                     <span class="project-author"><a href="@routes.UserApp.userInfo(project.owner)">@project.owner</a></span>
                     <span class="project-separator">/</span>
@@ -24,7 +24,7 @@
                     </span>
                     }
                 </div>
-                @if(project.isForkedFromOrigin()){
+                @if(project.isForkedFromOrigin){
                 <div class="project-origin">
                     <span class="project-origin-title">@Messages("fork.original")</span>
                     <a href="@routes.ProjectApp.project(project.originalProject.owner, project.originalProject.name)" class="project-origin-name">
app/views/project/list.scala.html
--- app/views/project/list.scala.html
+++ app/views/project/list.scala.html
@@ -32,7 +32,7 @@
                     <div style="float:left">
                         <div class="header">
                             <a href="@routes.ProjectApp.project(project.owner, project.name)" class="black">@project.name</a>
-                            @if(project.isForkedFromOrigin()){
+                            @if(project.isForkedFromOrigin){
                                 <span class="small-font blue-txt">
                                     <a class="origin-title" href='@routes.ProjectApp.project(project.originalProject.owner, project.originalProject.name)'>
                                         <i class="yobicon-split"></i>
@@ -43,7 +43,7 @@
 
                             @if(!project.isPublic){ <i class="yobicon-lock yobicon-small"></i> }
                             @for(label <- project.labels) {
-                                <a class="project-label @label.category.toLowerCase()" href="@routes.ProjectApp.projects()?labelIds=@label.id">@label.name</a>
+                                <a class="project-label @label.category.toLowerCase" href="@routes.ProjectApp.projects()?labelIds=@label.id">@label.name</a>
                             }
                         </div>
                         <div class="desc">
app/views/project/overview.scala.html
--- app/views/project/overview.scala.html
+++ app/views/project/overview.scala.html
@@ -7,22 +7,22 @@
 @import models.enumeration._
 
 @userPageUrlOnHistory(history:History) = @{
-	if(history.getUserPageUrl() != null) {
-		history.getUserPageUrl()
+	if(history.getUserPageUrl != null) {
+		history.getUserPageUrl
 	} else {
 		"#"
 	}
 }
 @userAvatarUrlOnHistory(history:History) = @{
-	if(history.getUserAvatarUrl() != null){
-		history.getUserAvatarUrl()
+	if(history.getUserAvatarUrl != null){
+		history.getUserAvatarUrl
 	} else {
 		routes.Assets.at("images/default-avatar-64.png")
 	}
 }
-@makeHistoryWho(history:History)  = {<a href="@userPageUrlOnHistory(history)" class="actor">@history.getWho()</a>}
-@makeHistoryHow(history:History)  = {<span class="whereis"><a href="@history.getUrl()" class="where">@history.getShortTitle()</a> <a href="@history.getUrl()" class="title">@history.getHow()</a></span> }
-@makeHistoryWhat(history:History) = {@Messages("project.history.type." + history.getWhat())}
+@makeHistoryWho(history:History)  = {<a href="@userPageUrlOnHistory(history)" class="actor">@history.getWho</a>}
+@makeHistoryHow(history:History)  = {<span class="whereis"><a href="@history.getUrl" class="where">@history.getShortTitle</a> <a href="@history.getUrl" class="title">@history.getHow</a></span> }
+@makeHistoryWhat(history:History) = {@Messages("project.history.type." + history.getWhat)}
 
 @projectLayout(message, project, utils.MenuType.PROJECT_HOME) {
 @projectMenu(project, utils.MenuType.PROJECT_HOME, "")
@@ -99,8 +99,8 @@
     	                                        ))
     	                                    </p>
     	                                    <p class="others" style="padding-left:0;">
-    	                                        <span class="date" style="margin-left:0;" title="@JodaDateUtil.getDateString(history.getWhen())">
-                                                    @agoString(JodaDateUtil.ago(history.getWhen()))
+	                                        <span class="date" style="margin-left:0;" title="@JodaDateUtil.getDateString(history.getWhen)">
+                                                    @agoString(JodaDateUtil.ago(history.getWhen))
                                                 </span>
     	                                    </p>
     	                                </div>
app/views/projectMenu.scala.html
--- app/views/projectMenu.scala.html
+++ app/views/projectMenu.scala.html
@@ -10,7 +10,7 @@
 }
 
 @getPullRequestURL(project:Project) = @{
-    if(project.isForkedFromOrigin()){
+    if(project.isForkedFromOrigin){
         routes.PullRequestApp.sentPullRequests(project.owner, project.name)
     } else {
         routes.PullRequestApp.pullRequests(project.owner, project.name)
app/views/site/issueList.scala.html
--- app/views/site/issueList.scala.html
+++ app/views/site/issueList.scala.html
@@ -16,7 +16,7 @@
         <li class="@isActiveTabMenu(State.CLOSED)"><a href="@routes.SiteApp.issueList()?state=@State.CLOSED.name.toLowerCase()">@Messages("issue.state." + State.CLOSED.name.toLowerCase())</a></li>
     </ul>
     <ul class="post-list-wrap">
-        @for(issue <- currentPage.getList()) {
+        @for(issue <- currentPage.getList) {
         @defining(User.findByLoginId(issue.authorLoginId)){ user =>
         <li class="row-fluid listitem">
             <a href="@routes.ProjectApp.project(issue.project.owner, issue.project.name)" class="avatar-wrap list-avatar">
@@ -61,7 +61,7 @@
 
     <script type="text/javascript">
     $(function(){
-        yobi.Pagination.update($("#pagination"), @currentPage.getTotalPageCount());
+        yobi.Pagination.update($("#pagination"), @currentPage.getTotalPageCount);
     });
     </script>    
 }
app/views/site/postList.scala.html
--- app/views/site/postList.scala.html
+++ app/views/site/postList.scala.html
@@ -8,7 +8,7 @@
         <h2 class="pull-left">@Messages("site.sidebar.postList")</h2>
     </div>
     <ul class="post-list-wrap">
-        @for(post <- currentPage.getList()) {
+        @for(post <- currentPage.getList) {
         @defining(User.findByLoginId(post.authorLoginId)){ user =>
         <li class="row-fluid listitem">
             <a href="@routes.ProjectApp.project(post.project.owner, post.project.name)" class="avatar-wrap list-avatar">
@@ -53,7 +53,7 @@
 
     <script type="text/javascript">
     $(function(){
-        yobi.Pagination.update($("#pagination"), @currentPage.getTotalPageCount());
+        yobi.Pagination.update($("#pagination"), @currentPage.getTotalPageCount);
     });
     </script>
 }
app/views/site/projectList.scala.html
--- app/views/site/projectList.scala.html
+++ app/views/site/projectList.scala.html
@@ -24,7 +24,7 @@
         </div>
     </div>
     <ul class="project-list-wrap">
-        @for(project <- currentPage.getList()) {
+        @for(project <- currentPage.getList) {
         <li class="row-fluid listitem">
             <div class="span5 listitem-col">
                 <a href="@routes.ProjectApp.project(project.owner,project.name)" class="avatar-wrap list-avatar">
@@ -77,7 +77,7 @@
             console.log($(this).data('href'));
         });
 
-        yobi.Pagination.update($("#pagination"), @currentPage.getTotalPageCount());
+        yobi.Pagination.update($("#pagination"), @currentPage.getTotalPageCount);
     });
     </script>
 }
app/views/site/userList.scala.html
--- app/views/site/userList.scala.html
+++ app/views/site/userList.scala.html
@@ -44,7 +44,7 @@
         }
     </div>
     <ul class="user-list-wrap">
-        @for(user <- currentPage.getList()) {
+        @for(user <- currentPage.getList) {
         <li class="row-fluid listitem">
             <div class="span3 listitem-col">
                 <a href="@routes.UserApp.userInfo(user.loginId)" class="avatar-wrap list-avatar">
@@ -115,7 +115,7 @@
             $('#alertDeletionWrap').modal('show');
         });
 
-        yobi.Pagination.update($("#pagination"), @currentPage.getTotalPageCount());
+        yobi.Pagination.update($("#pagination"), @currentPage.getTotalPageCount);
     });
     </script>
 }
app/views/user/edit.scala.html
--- app/views/user/edit.scala.html
+++ app/views/user/edit.scala.html
@@ -113,7 +113,7 @@
                                     <table class="table table-striped table-bordered">
                                     @for(notiType <- models.enumeration.EventType.notiTypes) {
                                         <tr>
-                                            <th>@notiType.getDescr()</th>
+                                            <th>@notiType.getDescr</th>
                                             <td>
                                                 <div class="switch" data-on-label="On" data-off-label="Off">
                                                     <input class="notiUpdate" data-href="@routes.WatchProjectApp.toggle(project.id, notiType.name())" type="checkbox" data-toggle="switch" @if(UserProjectNotification.isEnabledNotiType(notiMap, project, notiType)){ checked="checked" }>
app/views/user/partial_projectlist.scala.html
--- app/views/user/partial_projectlist.scala.html
+++ app/views/user/partial_projectlist.scala.html
@@ -14,7 +14,7 @@
                 <a href="@routes.UserApp.userInfo(project.owner)" class="owner-name-small">@project.owner</a> / <a href="@routes.ProjectApp.project(project.owner, project.name)" class="project-name">@project.name</a>
                 @if(!project.isPublic){ <i class="yobicon-lock yobicon-small"></i> }
             </div>
-            @if(project.isForkedFromOrigin()){
+            @if(project.isForkedFromOrigin){
             <div class="forked label">
                 <i class="yobicon-split yobicon-white vmiddle"></i>
                 @Messages("fork.original")
@@ -24,13 +24,13 @@
             }
             <div class="desc tag">
                 @for(label <- project.labels) {
-                <span class="project-label @label.category.toLowerCase()">@label.name</span>
+                <span class="project-label @label.category.toLowerCase">@label.name</span>
                 }
             </div>
             <div class="desc">@project.overview</div>
             <div class="name-tag">
                 @Messages("project.create") <strong>@JodaDateUtil.momentFromNow(project.createdDate, lang.language)</strong>
-                @if(project.lastPushedDateAgo() != null) {, @Messages("project.codeUpdate") <strong>@JodaDateUtil.momentFromNow(project.lastPushedDate.getTime(), lang.language)}</strong>
+                @if(project.lastPushedDateAgo() != null) {, @Messages("project.codeUpdate") <strong>@JodaDateUtil.momentFromNow(project.lastPushedDate.getTime, lang.language)}</strong>
             </div>
         </div>
     </div>
Add a comment
List