Skip to content

Commit fbe6c6d

Browse files
committed
(refs #12)Remove PARENT_USER_NAME and PARENT_REPOSITORY_NAME from GIST table
1 parent ba35766 commit fbe6c6d

File tree

6 files changed

+25
-31
lines changed

6 files changed

+25
-31
lines changed

src/main/resources/update/1_3.sql

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,2 @@
11
ALTER TABLE GIST ADD COLUMN ORIGIN_USER_NAME VARCHAR(100);
22
ALTER TABLE GIST ADD COLUMN ORIGIN_REPOSITORY_NAME VARCHAR(100);
3-
ALTER TABLE GIST ADD COLUMN PARENT_USER_NAME VARCHAR(100);
4-
ALTER TABLE GIST ADD COLUMN PARENT_REPOSITORY_NAME VARCHAR(100);

src/main/scala/gitbucket/gist/controller/GistController.scala

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -298,7 +298,7 @@ trait GistControllerBase extends ControllerBase {
298298
val originRepoName = gist.originRepositoryName.getOrElse(gist.repositoryName)
299299

300300
registerGist(loginAccount.userName, repoName, gist.isPrivate, gist.title, gist.description,
301-
Some(originUserName), Some(originRepoName), Some(userName), Some(repoName))
301+
Some(originUserName), Some(originRepoName))
302302

303303
// Clone repository
304304
JGitUtil.cloneRepository(

src/main/scala/gitbucket/gist/model/Gist.scala

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -16,9 +16,7 @@ trait GistComponent { self: gitbucket.core.model.Profile =>
1616
val updatedDate = column[java.util.Date]("UPDATED_DATE")
1717
val originUserName = column[String]("ORIGIN_USER_NAME")
1818
val originRepositoryName = column[String]("ORIGIN_REPOSITORY_NAME")
19-
val parentUserName = column[String]("PARENT_USER_NAME")
20-
val parentRepositoryName = column[String]("PARENT_REPOSITORY_NAME")
21-
def * = (userName, repositoryName, isPrivate, title, description, registeredDate, updatedDate, originUserName.?, originRepositoryName.?, parentUserName.?, parentRepositoryName.?) <> (Gist.tupled, Gist.unapply)
19+
def * = (userName, repositoryName, isPrivate, title, description, registeredDate, updatedDate, originUserName.?, originRepositoryName.?) <> (Gist.tupled, Gist.unapply)
2220
}
2321
}
2422

@@ -31,9 +29,7 @@ case class Gist(
3129
registeredDate: java.util.Date,
3230
updatedDate: java.util.Date,
3331
originUserName: Option[String],
34-
originRepositoryName: Option[String],
35-
parentUserName: Option[String],
36-
parentRepositoryName: Option[String]
32+
originRepositoryName: Option[String]
3733
)
3834

3935

src/main/scala/gitbucket/gist/service/GistService.scala

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -40,10 +40,9 @@ trait GistService {
4040
Gists.filter(t => (t.originUserName === userName.bind) && (t.originRepositoryName === repositoryName.bind)).sortBy(_.userName).list
4141

4242
def registerGist(userName: String, repositoryName: String, isPrivate: Boolean, title: String, description: String,
43-
originUserName: Option[String] = None, originRepositoryName: Option[String] = None,
44-
parentUserName: Option[String] = None, parentRepositoryName: Option[String] = None)(implicit s: Session): Unit =
43+
originUserName: Option[String] = None, originRepositoryName: Option[String] = None)(implicit s: Session): Unit =
4544
Gists.insert(Gist(userName, repositoryName, isPrivate, title, description, new java.util.Date(), new java.util.Date(),
46-
originUserName, originRepositoryName, parentUserName, parentRepositoryName))
45+
originUserName, originRepositoryName))
4746

4847
def updateGist(userName: String, repositoryName: String, title: String, description: String)(implicit s: Session): Unit =
4948
Gists

src/main/twirl/gitbucket/gist/header.scala.html

Lines changed: 12 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -16,20 +16,19 @@
1616
<a href="@path/gist/@gist.userName/@gist.repositoryName/edit" class="btn btn-small">Edit</a>
1717
<a href="@path/gist/@gist.userName/@gist.repositoryName/delete" class="btn btn-small" id="delete">Delete</a>
1818
}
19-
<div class="input-prepend" style="margin-bottom: 0px;">
20-
@if(loginAccount.isEmpty){
21-
<a title="You must be signed in to fork a repository" href="@path/signin" class="btn btn-small">Fork</a>
22-
} else {
23-
<a id="fork-link" href="javascript:void(0);" class="btn btn-small">Fork</a>
19+
@if(gist.userName.isEmpty){
20+
<div class="input-prepend" style="margin-bottom: 0px;">
21+
@if(loginAccount.isEmpty){
22+
<a title="You must be signed in to fork a repository" href="@path/signin" class="btn btn-small">Fork</a>
23+
} else {
24+
<a id="fork-link" href="javascript:void(0);" class="btn btn-small">Fork</a>
25+
}
26+
<span class="add-on count"><a href="@path/gist/@gist.userName/@gist.repositoryName/forks">@forkedCount</a></span>
27+
</div>
28+
@if(loginAccount.isDefined){
29+
<form id="fork-form" method="post" action="@path/gist/@gist.userName/@gist.repositoryName/fork">
30+
</form>
2431
}
25-
<span class="add-on count"><a href="@path/gist/@gist.userName/@gist.repositoryName/forks">@forkedCount</a></span>
26-
</div>
27-
@if(loginAccount.isDefined){
28-
<form id="fork-form" method="post" action="@path/gist/@gist.userName/@gist.repositoryName/fork">
29-
@*
30-
<input type="hidden" name="account" value="@loginAccount.get.userName"/>
31-
*@
32-
</form>
3332
}
3433
</div>
3534
</div>

src/main/twirl/gitbucket/gist/menu.scala.html

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -39,12 +39,14 @@
3939
dataURI("images/menu-revisions-active.png"),
4040
dataURI("images/menu-revisions.png"),
4141
"Revision")
42-
@menuitem(
43-
"forks",
44-
s"${gist.userName}/${gist.repositoryName}/forks",
45-
dataURI("images/menu-forks-active.png"),
46-
dataURI("images/menu-forks.png"),
47-
"Forks")
42+
@if(gist.originUserName.isEmpty){
43+
@menuitem(
44+
"forks",
45+
s"${gist.userName}/${gist.repositoryName}/forks",
46+
dataURI("images/menu-forks-active.png"),
47+
dataURI("images/menu-forks.png"),
48+
"Forks")
49+
}
4850
<li style="height: 12px"><div class="gradient pull-left" style="height: 12px"></div></li>
4951
</ul>
5052
<div class="small">

0 commit comments

Comments
 (0)