Fix #183

diff --git a/modules/middleware/repo.go b/modules/middleware/repo.go
index bfb1e69da86ceaefd2a2a93a37584d793542158d..cd4353912245d280202f384367e0e60e7a9f333c 100644 (file)
--- a/modules/middleware/repo.go
+++ b/modules/middleware/repo.go
@@ -99,6 +99,11 @@ func RepoAssignment(redirect bool, args ...bool) martini.Handler {
                        return
                }
 
+               // Check if the mirror repository owner(mirror repository doesn't have access).
+               if !ctx.Repo.IsOwner && repo.OwnerId == ctx.User.Id {
+                       ctx.Repo.IsOwner = true
+               }
+
                // Check access.
                if repo.IsPrivate && !ctx.Repo.IsOwner {
                        if ctx.User == nil {

diff --git a/routers/repo/setting.go b/routers/repo/setting.go
index 89c7cb486e6ad1a8280f07512391a81e02d4ec87..fa4973ecd6e95849c8fc76f9c0f4aac57b8011e3 100644 (file)
--- a/routers/repo/setting.go
+++ b/routers/repo/setting.go
@@ -83,6 +83,9 @@ func SettingPost(ctx *middleware.Context, form auth.RepoSettingForm) {
                if len(ctx.Repo.Repository.Name) == 0 || ctx.Repo.Repository.Name != ctx.Query("repository") {
                        ctx.RenderWithErr("Please make sure you entered repository name is correct.", "repo/setting", nil)
                        return
+               } else if ctx.Repo.Repository.IsMirror {
+                       ctx.Error(404)
+                       return
                }
 
                newOwner := ctx.Query("owner")

diff --git a/routers/user/home.go b/routers/user/home.go
index b19f3ff4aca7793996ce703b95e0cd6ad898af56..a7a36b4bd70d4f94214fbf95d6a1fffee35ed533 100644 (file)
--- a/routers/user/home.go
+++ b/routers/user/home.go
@@ -235,8 +235,7 @@ func Issues(ctx *middleware.Context) {
                        return
                }
 
-               err = issues[i].Repo.GetOwner()
-               if err != nil {
+               if err = issues[i].Repo.GetOwner(); err != nil {
                        ctx.Handle(500, "user.Issues(GetOwner)", err)
                        return
                }

diff --git a/templates/repo/setting.tmpl b/templates/repo/setting.tmpl
index f0f041dee897016b7cf7e92f97ad511df0bc2b91..99835640fcc3acfca13150df3815a2f0d3884986 100644 (file)
--- a/templates/repo/setting.tmpl
+++ b/templates/repo/setting.tmpl
@@ -88,6 +88,7 @@
                 Danger Zone
             </div>
             
+            {{if not .Repository.IsMirror}}
             <div class="panel-body">
                 <button type="button" class="btn btn-default pull-right" href="#transfer-repository-modal" data-toggle="modal">
                     Transfer ownership 
@@ -137,6 +138,8 @@
                     </div>
                 </div>
             </div>
+            {{end}}
+            
             <hr>
             <div class="panel-body">
                 <button type="button" class="btn btn-default pull-right" href="#delete-repository-modal" data-toggle="modal">