kallithea Changeset - 2d216fd7d5a2

Changeset - 2d216fd7d5a2

Parent rev.

Child rev.

[Not reviewed]

default

0 5 0

Søren Løvborg - 9 years ago 2016-09-19 14:41:11
sorenl@unity3d.com

db: rename UserFollowing.follows_repo_id to follows_repository_id

Since the relationship is 'follows_repository', rename the column to be
'follows_repository_id', not 'follows_repo_id'. This also makes the
Python column name match the actual database column name.

(The inconsistency dates back to early RhodeCode days.)

5 files changed with 11 insertions and 11 deletions:

kallithea/controllers/journal.py

kallithea/model/db.py

kallithea/model/scm.py

kallithea/public/js/base.js

kallithea/tests/functional/test_journal.py

0 comments (0 inline, 0 general)

kallithea/controllers/journal.py

➞

Show inline comments

@@ @@ -78,8 +78,8 @@ class JournalController(BaseController): @@
         return groups
     def _get_journal_data(self, following_repos):
         repo_ids = [x.follows_repo_id for x in following_repos
                     if x.follows_repo_id is not None]
         repo_ids = [x.follows_repository_id for x in following_repos
                     if x.follows_repository_id is not None]
         user_ids = [x.follows_user_id for x in following_repos
                     if x.follows_user_id is not None]
@@ @@ -311,7 +311,7 @@ class JournalController(BaseController): @@
                 log.error(traceback.format_exc())
                 raise HTTPBadRequest()
-        repo_id = request.POST.get('follows_repo_id')
+        repo_id = request.POST.get('follows_repository_id')
         if repo_id:
             try:
                 self.scm_model.toggle_following_repo(repo_id,

kallithea/model/db.py

➞

Show inline comments

@@ @@ -1021,7 +1021,7 @@ class Repository(Base, BaseDbModel): @@
     stats = relationship('Statistics', cascade='all', uselist=False)
     followers = relationship('UserFollowing',
-                             primaryjoin='UserFollowing.follows_repo_id==Repository.repo_id',
+                             primaryjoin='UserFollowing.follows_repository_id==Repository.repo_id',
                              cascade='all')
     extra_fields = relationship('RepositoryField',
                                 cascade="all, delete-orphan")
@@ @@ -2036,7 +2036,7 @@ class UserFollowing(Base, BaseDbModel): @@
     user_following_id = Column(Integer(), primary_key=True)
     user_id = Column(Integer(), ForeignKey('users.user_id'), nullable=False)
-    follows_repo_id = Column("follows_repository_id", Integer(), ForeignKey('repositories.repo_id'), nullable=True)
+    follows_repository_id = Column(Integer(), ForeignKey('repositories.repo_id'), nullable=True)
     follows_user_id = Column(Integer(), ForeignKey('users.user_id'), nullable=True)
     follows_from = Column(DateTime(timezone=False), nullable=False, default=datetime.datetime.now)
@@ @@ -2047,7 +2047,7 @@ class UserFollowing(Base, BaseDbModel): @@
     @classmethod
     def get_repo_followers(cls, repo_id):
-        return cls.query().filter(cls.follows_repo_id == repo_id)
+        return cls.query().filter(cls.follows_repository_id == repo_id)
 class CacheInvalidation(Base, BaseDbModel):

kallithea/model/scm.py

➞

Show inline comments

@@ @@ -241,7 +241,7 @@ class ScmModel(BaseModel): @@
     def toggle_following_repo(self, follow_repo_id, user_id):
         f = self.sa.query(UserFollowing) \
-            .filter(UserFollowing.follows_repo_id == follow_repo_id) \
+            .filter(UserFollowing.follows_repository_id == follow_repo_id) \
             .filter(UserFollowing.user_id == user_id).scalar()
         if f is not None:
@@ @@ -258,7 +258,7 @@ class ScmModel(BaseModel): @@
         try:
             f = UserFollowing()
             f.user_id = user_id
-            f.follows_repo_id = follow_repo_id
+            f.follows_repository_id = follow_repo_id
             self.sa.add(f)
             action_logger(UserTemp(user_id),

kallithea/public/js/base.js

➞

Show inline comments

@@ @@ -451,8 +451,8 @@ var _onSuccessFollow = function(target){ @@
+    }
+}
 var toggleFollowingRepo = function(target, follows_repo_id){
     var args = 'follows_repo_id=' + follows_repo_id;
 var toggleFollowingRepo = function(target, follows_repository_id){
     var args = 'follows_repository_id=' + follows_repository_id;
     args += '&amp;_authentication_token=' + _authentication_token;
     $.post(TOGGLE_FOLLOW_URL, args, function(data){
             _onSuccessFollow(target);

kallithea/tests/functional/test_journal.py

➞

Show inline comments

@@ @@ -23,7 +23,7 @@ class TestJournalController(TestControll @@
+#
 #        response = self.app.post(url(controller='journal',
 #                                     action='toggle_following'),
-#                                     {'follows_repo_id':repo.repo_id})
+#                                     {'follows_repository_id':repo.repo_id})
     def test_start_following_repository(self):
         self.log_user()

0 comments (0 inline, 0 general)