|
Notes |
|
|
(0018674)
|
|
RedRat
|
20-04-2019 13:37
(edited on: 20-04-2019 13:45) |
|
Пара уточнений:
1. При переносе любой категории на уровень с одноимённой, подкатегории из последней попадают в переносимую, а точки остаются на месте.
2. Данный баг стабильно повторяется как на SQLite, так и на SML базах данных.
По всей видимости, проблема заключается в том, что в БД все подкатегории связаны с родительскими категориями только параметром пути, а не ссылкой на индекс родителя, поэтому при наличии двух или более одноимённых категорий программа просто не знает, какие подкатегории куда совать. Ошибка дизайна БД?
|
|
|
|
(0018675)
|
|
RedRat
|
|
20-04-2019 13:51
|
|
Для быстрой демонстрации бага сделал маленькую базу в SML и приложил её к этому репорту. Достаточно открыть её и переименовать категорию Cat3 в Cat1 - всё сами увидите.
До полноценного решения этой проблемы предлагаю в ночных сборках пока запретить создание одноимённых категорий, чтобы избежать непредвиденной потери данных. |
|
|
|
(0018676)
|
|
zed
|
|
20-04-2019 14:10
|
|
|
В SML так было by design, а SQLite и прочие унаследовали поведение. Поэтому, запрет создания одноимённых категорий - по-моему единственный способ решения этого бага. |
|
|
|
(0018684)
|
|
RedRat
|
|
22-04-2019 11:18
|
|
|
Либо, как вариант, при появлении одноимённых категорий на одном уровне объединять их в одну с переносом всех точек и подкатегорий. |
|
|
|
|
|
Предлагаю фикс попроще. При удалении категории, проверять есть ли с еще категории с таким же именем. И если есть, то просто не запускать удаления дочерних категорий. |
|
|
|
(0018726)
|
|
zed
|
|
04-06-2019 07:14
|
|
А может добавить в интерфейс поле NameInternal, сделать его уникальным (Name + ID) и использовать его, а для гуя оставить Name? Придётся залезть в реализацию SML и ORM (без изменения схемы), зато должно получиться надёжно.
Кроме того, в интерфейсе меток/категорий метод GetByName надо переделать так, чтобы он возвращал список. GetMarkByName по сути используется только в одном месте, при импорте меток, когда установлен флаг не импортировать, если метка с таким именем существует. И результат функции вообще не используется, а только проверяется на nil. А вот GetCategoryByName - сплошь и всюду. |
|
|
|
(0018727)
|
|
zed
|
|
04-06-2019 07:25
|
|
|
А, нет. Надо менять схему и добавлять ParentID. |
|
|
|
(0018730)
|
|
zed
|
|
04-06-2019 07:53
|
|
Значит, буду делать так:
Методы GetXXXByName переименую в GetFirstXXXByName, поскольку списки одноимённых меток/категорий нам пока нигде не нужны, но имя метода будет явно говорить, что могут быть и Next.
Добавлю методы GetXXXByNameCount - который будет возвращать число одноимённых меток, чтобы где надо, можно было просто проверить есть/нету или есть ли дубликаты.
В гуе сделаю запрет на создание (в том числе и методом переименования) одноимённых категорий и сделаю запрет удаления категории, если есть одноимённая. Для удаления такой категории, её надо будет сперва переименовать. |
|
|
|
|
> В гуе сделаю запрет на создание (в том числе и методом переименования) одноимённых категорий и сделаю запрет удаления категории, если есть одноимённая. Для удаления такой категории, её надо будет сперва переименовать.
Старые базы все еще будут жить с дублирующимися метками. Но хозяин - барин. |
|
|
|
(0018732)
|
|
zed
|
|
04-06-2019 08:27
|
|
|
Одинаковые имена - не проблема БД, ведь если изменять схему, то имена так и останутся одинаковыми. По хорошему, да, схему надо менять, но это слишком калечащее изменение для поддержки не очень частого случая. |
|
|
|
(0018733)
|
|
zed
|
|
04-06-2019 12:31
|
|
|
Сделал, тестируйте следующую ночную версию. |
|
Test.db3 (73,728) 16-04-2019 14:33