remove dup language set select, add permission to dataset edit.

custom/templates/base/footer_content.tmpl

@@ -18,15 +18,6 @@
 					<div class="column ui vertical text menu">
 						<div class="header item">帮助</div>
 						<a href="https://docs.gitea.io/zh-cn/" class="item">帮助文档</a>
-						<div class="ui language bottom floating slide up dropdown link item">
-							<i class="world icon"></i>
-							<div class="text">{{.LangName}}</div>
-							<div class="menu">
-								{{range .AllLangs}}
-									<a lang="{{.Lang}}" class="item {{if eq $.Lang .Lang}}active selected{{end}}" href="{{if eq $.Lang .Lang}}#{{else}}{{$.Link}}?lang={{.Lang}}{{end}}">{{.Name}}</a>
-								{{end}}
-							</div>
-						</div>
 						{{template "custom/extra_links_footer" .}}
 					</div>
 				</div>

routers/dataset/dataset.go

@@ -1,6 +1,7 @@
 package dataset
 
 import (
+	"errors"
 	"strings"
 
 	"code.gitea.io/gitea/models"
@@ -25,7 +26,7 @@ type ListOptions struct {
 	Page     int // start from 1
 }
 
-func MyList(ctx *context.Context) {
+func List(ctx *context.Context) {
 	ctx.Data["PageIsDatasets"] = true
 
 	var (
@@ -39,6 +40,7 @@ func MyList(ctx *context.Context) {
 	if page <= 0 {
 		page = 1
 	}
+	ctx.Data["SortType"] = ctx.Query("sort")
 	switch ctx.Query("sort") {
 	case "newest":
 		orderBy = models.SearchOrderByNewest
@@ -54,16 +56,6 @@ func MyList(ctx *context.Context) {
 		orderBy = models.SearchOrderByAlphabetically
 	case "reversesize":
 		orderBy = models.SearchOrderBySizeReverse
-	case "size":
-		orderBy = models.SearchOrderBySize
-	case "moststars":
-		orderBy = models.SearchOrderByStarsReverse
-	case "feweststars":
-		orderBy = models.SearchOrderByStars
-	case "mostforks":
-		orderBy = models.SearchOrderByForksReverse
-	case "fewestforks":
-		orderBy = models.SearchOrderByForks
 	default:
 		ctx.Data["SortType"] = "recentupdate"
 		orderBy = models.SearchOrderByRecentUpdated
@@ -100,7 +92,6 @@ func MyList(ctx *context.Context) {
 
 	ctx.Data["datasets"] = datasets
 	ctx.Data["datasetsCount"] = count
-	log.Debug("[dataset] mylist...\n")
 	ctx.HTML(200, tplDataSet)
 }
 
@@ -171,7 +162,7 @@ func Show(ctx *context.Context) {
 		return
 	}
 
-	isOwner := dataset.UserID == user.ID
+	isOwner := (ctx.User != nil && dataset.UserID == user.ID)
 
 	ctx.Data["dataset"] = dataset
 	ctx.Data["IsOwner"] = isOwner
@@ -189,6 +180,11 @@ func EditDataset(ctx *context.Context) {
 		ctx.NotFound("GetDataset", err)
 		return
 	}
+	isOwner := (ctx.User != nil && rel.UserID == ctx.User.ID)
+	if !isOwner {
+		ctx.NotFound("403", errors.New("not owner"))
+		return
+	}
 
 	ctx.Data["ID"] = rel.ID
 	ctx.Data["title"] = rel.Title
@@ -209,6 +205,11 @@ func EditDatasetPost(ctx *context.Context, form auth.EditDatasetForm) {
 		ctx.ServerError("GetDataset", err)
 		return
 	}
+	isOwner := (ctx.User != nil && rel.UserID == ctx.User.ID)
+	if !isOwner {
+		ctx.NotFound("403", errors.New("not owner"))
+		return
+	}
 	ctx.Data["ID"] = rel.ID
 	ctx.Data["title"] = rel.Title
 	ctx.Data["description"] = rel.Description

routers/routes/routes.go

@@ -1007,7 +1007,7 @@ func RegisterRoutes(m *macaron.Macaron) {
 
 	// DataSet
 	m.Group("/datasets", func() {
-		m.Get("", dataset.MyList)
+		m.Get("", dataset.List)
 		m.Get("/create", dataset.Create)
 		m.Post("/create", bindIgnErr(auth.CreateDatasetForm{}), dataset.CreatePost)
 		m.Post("/delete", dataset.Delete)

templates/datasets/show.tmpl

@@ -32,7 +32,9 @@
             </div>
             <div class="ui twelve wide column detail">
               <h3><a href="">{{.dataset.Title}}</a>
+              {{if .isOwner}}
               <small>(<a href="/datasets/edit/{{.dataset.ID}}" rel="nofollow">{{$.i18n.Tr "dataset.edit"}}</a>)</small>
+              {{end}}
               </h3>
               <div class="markdown desc">
                 <p>{{.dataset.Description}}</p>