Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug: Workspace почему-то должен иметь уникальное имя #264

Closed
bazilval opened this issue May 17, 2024 · 14 comments
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@bazilval
Copy link
Collaborator

Summary

На данный момент workspace не может быть создан с одним и тем же именем, что кажется довольно странным, учитывая, что пользователь может захотеть назвать свою доску как угодно и не должен учитывать названия досок у всех остальных. Для уникальности есть id.

При чём попытка создания доски с тем же названием, никак не перехватывается

Reproduction steps

1. Создать доску с любым названием
2. Создать ещё одну с тем же названием
3. Словить необработанное исключение

Expected result

Доска создаётся без проблем

Actual result

Выводится необработанная ошибка на сервере

Browsers

No response

OS

No response

@bazilval bazilval added the bug Something isn't working label May 17, 2024
@fey fey added the good first issue Good for newcomers label May 17, 2024
@kitdim
Copy link
Contributor

kitdim commented May 30, 2024

возьму

@kitdim
Copy link
Contributor

kitdim commented Jun 4, 2024

Я багу поправил, всё делал через миграции. Локально всё запускается, и вортспейсы с одинаковым названием создавать можно, но чо-то при билдинге такая вот ошибка почти на всех тестах, как я понимаю связана с миграцией:
AccountRepositoryIT > getAccountByEmail(String) > [3] email=test103@gmail.com FAILED java.lang.IllegalStateException at DefaultCacheAwareContextLoaderDelegate.java:142 Caused by: org.springframework.beans.factory.BeanCreationException at AbstractAutowireCapableBeanFactory.java:1762 Caused by: liquibase.exception.UnexpectedLiquibaseException at SpringLiquibase.java:304 Caused by: liquibase.exception.CommandExecutionException at CommandScope.java:253 Caused by: liquibase.exception.LiquibaseException at ChangeLogIterator.java:151 Caused by: liquibase.exception.MigrationFailedException at ChangeSet.java:797 Caused by: liquibase.exception.DatabaseException at JdbcExecutor.java:470 Caused by: org.postgresql.util.PSQLException at QueryExecutorImpl.java:2676

@bazilval
Copy link
Collaborator Author

bazilval commented Jun 4, 2024

пушни на репу, чтобы посмотреть что там у тебя

@kitdim
Copy link
Contributor

kitdim commented Jun 4, 2024

пушни на репу, чтобы посмотреть что там у тебя

#270

@bazilval
Copy link
Collaborator Author

bazilval commented Jun 8, 2024

@kitdim привет!
А что за constraintName такое у тебя? "uk_br8l0q43h1ygdohbp4htocj3h"

Я вот в самом первом changeSet нашёл такой констрейнт на уникальность имени доски

    <changeSet id="20211904104233167-3" author="system">
        <addUniqueConstraint columnNames="name" constraintName="UC_WORKSPACE_NAME" tableName="workspace" />
    </changeSet>

Может попробовать это имя подставить?

@kitdim
Copy link
Contributor

kitdim commented Jun 9, 2024

@bazilval привет)
А я его в бд в схеме нашел.
Да ты прав, там плюс добавил отключение workspace_name_key.
После этого всё ок, ща изменение запулю

@kitdim
Copy link
Contributor

kitdim commented Jun 9, 2024

@bazilval cпасибо!

@bazilval
Copy link
Collaborator Author

@kitdim а что за workspace_name_key?

@kitdim
Copy link
Contributor

kitdim commented Jun 10, 2024

Похоже констрейнтов для name было два. Если отключить только UC_WORKSPACE_NAME, последуется провера на workspace_name_key, и появляется такая же ошибка как с UC_WORKSPACE_NAME
image

@bazilval
Copy link
Collaborator Author

@kitdim хм, а ты не нашёл в какой момент его добавляли и зачем?

@kitdim
Copy link
Contributor

kitdim commented Jun 11, 2024

@bazilval добавили в этом коммите
сама миграция вот

@bazilval
Copy link
Collaborator Author

@kitdim странно как-то, да?
вроде в самом коммите добавляют уникальность URL
а в миграции добавляют дублирующую уникальность имени

@kitdim
Copy link
Contributor

kitdim commented Jun 13, 2024

@bazilval ага. Возможно это сделано специально, чтобы в будующем поправили)

fey pushed a commit that referenced this issue Jun 14, 2024
* remove unique constrain to name workspace

* remove unique constrain to name workspace
@kitdim
Copy link
Contributor

kitdim commented Jun 16, 2024

@fey привет! Закрываем?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

3 participants