diff --git a/kallithea/controllers/admin/gists.py b/kallithea/controllers/admin/gists.py --- a/kallithea/controllers/admin/gists.py +++ b/kallithea/controllers/admin/gists.py @@ -79,17 +79,17 @@ class GistsController(BaseController): # MY private if c.show_private and not c.show_public: gists = gists.filter(Gist.gist_type == Gist.GIST_PRIVATE) \ - .filter(Gist.gist_owner == c.authuser.user_id) + .filter(Gist.owner_id == c.authuser.user_id) # MY public elif c.show_public and not c.show_private: gists = gists.filter(Gist.gist_type == Gist.GIST_PUBLIC) \ - .filter(Gist.gist_owner == c.authuser.user_id) + .filter(Gist.owner_id == c.authuser.user_id) # MY public+private elif c.show_private and c.show_public: gists = gists.filter(or_(Gist.gist_type == Gist.GIST_PUBLIC, Gist.gist_type == Gist.GIST_PRIVATE)) \ - .filter(Gist.gist_owner == c.authuser.user_id) + .filter(Gist.owner_id == c.authuser.user_id) # default show ALL public gists if not c.show_public and not c.show_private: @@ -153,7 +153,7 @@ class GistsController(BaseController): @NotAnonymous() def delete(self, gist_id): gist = GistModel().get_gist(gist_id) - owner = gist.gist_owner == c.authuser.user_id + owner = gist.owner_id == c.authuser.user_id if h.HasPermissionAny('hg.admin')() or owner: GistModel().delete(gist) Session().commit() diff --git a/kallithea/controllers/api/api.py b/kallithea/controllers/api/api.py --- a/kallithea/controllers/api/api.py +++ b/kallithea/controllers/api/api.py @@ -2380,7 +2380,7 @@ class ApiController(JSONRPCController): """ gist = get_gist_or_error(gistid) if not HasPermissionAny('hg.admin')(): - if gist.gist_owner != self.authuser.user_id: + if gist.owner_id != self.authuser.user_id: raise JSONRPCError('gist `%s` does not exist' % (gistid,)) return gist.get_api_data() @@ -2408,7 +2408,7 @@ class ApiController(JSONRPCController): gists = [] _gists = Gist().query() \ .filter(or_(Gist.gist_expires == -1, Gist.gist_expires >= time.time())) \ - .filter(Gist.gist_owner == user_id) \ + .filter(Gist.owner_id == user_id) \ .order_by(Gist.created_on.desc()) for gist in _gists: gists.append(gist.get_api_data()) @@ -2509,7 +2509,7 @@ class ApiController(JSONRPCController): """ gist = get_gist_or_error(gistid) if not HasPermissionAny('hg.admin')(): - if gist.gist_owner != self.authuser.user_id: + if gist.owner_id != self.authuser.user_id: raise JSONRPCError('gist `%s` does not exist' % (gistid,)) try: diff --git a/kallithea/model/db.py b/kallithea/model/db.py --- a/kallithea/model/db.py +++ b/kallithea/model/db.py @@ -2525,7 +2525,7 @@ class Gist(Base, BaseDbModel): gist_id = Column(Integer(), primary_key=True) gist_access_id = Column(Unicode(250), nullable=False) gist_description = Column(UnicodeText(), nullable=False) - gist_owner = Column('user_id', Integer(), ForeignKey('users.user_id'), nullable=False) + owner_id = Column('user_id', Integer(), ForeignKey('users.user_id'), nullable=False) gist_expires = Column(Float(53), nullable=False) gist_type = Column(Unicode(128), nullable=False) created_on = Column(DateTime(timezone=False), nullable=False, default=datetime.datetime.now) diff --git a/kallithea/model/gist.py b/kallithea/model/gist.py --- a/kallithea/model/gist.py +++ b/kallithea/model/gist.py @@ -118,7 +118,7 @@ class GistModel(BaseModel): gist = Gist() gist.gist_description = description gist.gist_access_id = gist_id - gist.gist_owner = owner.user_id + gist.owner_id = owner.user_id gist.gist_expires = gist_expires gist.gist_type = safe_unicode(gist_type) self.sa.add(gist) diff --git a/kallithea/templates/admin/gists/show.html b/kallithea/templates/admin/gists/show.html --- a/kallithea/templates/admin/gists/show.html +++ b/kallithea/templates/admin/gists/show.html @@ -50,7 +50,7 @@ %endif - %if h.HasPermissionAny('hg.admin')() or c.gist.gist_owner == c.authuser.user_id: + %if h.HasPermissionAny('hg.admin')() or c.gist.owner_id == c.authuser.user_id:
${h.form(url('gist_delete', gist_id=c.gist.gist_id))} ${h.submit('remove_gist', _('Delete'),class_="btn btn-danger btn-xs",onclick="return confirm('"+_('Confirm to delete this Gist')+"');")} @@ -59,7 +59,7 @@ %endif
## only owner should see that - %if h.HasPermissionAny('hg.admin')() or c.gist.gist_owner == c.authuser.user_id: + %if h.HasPermissionAny('hg.admin')() or c.gist.owner_id == c.authuser.user_id: ${h.link_to(_('Edit'),h.url('edit_gist', gist_id=c.gist.gist_access_id),class_="btn btn-default btn-xs")} %endif ${h.link_to(_('Show as Raw'),h.url('formatted_gist', gist_id=c.gist.gist_access_id, format='raw'),class_="btn btn-default btn-xs")}