Kallithea changeset: 8188:f8314738030a

[Mads Kiilerich]

Hi

Thanks - we have queued that fix up after it was mentioned on IRC.

The extra spaces have been there "forever" without anybody reporting it, 
so very weird that you see it now. It would be nice to understand that. 
Is there anything special in which packages or database you use?

/Mads



On 4/9/20 10:45 PM, Андрей Макаров wrote:
> Today I was be trying to install fresh kallithea.
> The setup was failed on
>    kallithea-cli db-create -c my.ini --user=XXX --password=YYY 
> --email=mka-at-mailru at mail.ru --repos=/ZZZ/
> with error
> sqlalchemy.exc.InvalidRequestError: When initializing mapper mapped 
> class UserGroup->users_groups, expression 'UserUserGroupToPerm ' 
> failed to locate a name ('UserUserGroupToPerm '). If this is a class 
> name, consider adding this relationship() to the <class 
> 'kallithea.model.db.UserGroup'> class after both dependent classes 
> have been defined.
> Please, check 'UserUserGroupToPerm ' names — is there extra spaces 
> after name?
> diff -r f8314738030a kallithea/model/db.py
> --- a/kallithea/model/db.py     Sun Nov 03 16:39:59 2019 +0100
> +++ b/kallithea/model/db.py     Thu Apr 09 23:42:46 2020 +0300
> @@ -820,8 +820,8 @@
>      users_group_to_perm = relationship('UserGroupToPerm', cascade='all')
>      users_group_repo_to_perm = relationship('UserGroupRepoToPerm', 
> cascade='all')
>      users_group_repo_group_to_perm = 
> relationship('UserGroupRepoGroupToPerm', cascade='all')
> -    user_user_group_to_perm = relationship('UserUserGroupToPerm ', 
> cascade='all')
> -    user_group_user_group_to_perm = 
> relationship('UserGroupUserGroupToPerm ', 
> primaryjoin="UserGroupUserGroupToPerm.target_user_group_id==UserGroup.users_group_id", 
> cascade='all')
> +    user_user_group_to_perm = relationship('UserUserGroupToPerm', 
> cascade='all')
> +    user_group_user_group_to_perm = 
> relationship('UserGroupUserGroupToPerm', 
> primaryjoin="UserGroupUserGroupToPerm.target_user_group_id==UserGroup.users_group_id", 
> cascade='all')
>
>      owner = relationship('User')
> P.S. I’m sorry… I'm too lazy to register and create pull requests for 
> a few of spaces.
> -----------
> Regards,
> Andrey