search

LeKaitoW / raox-game5

game5 plugin for raox
0 stars 3 forks source link

Избавиться от ворнингов #4

Closed aurusov closed 9 years ago

aurusov commented 9 years ago

Налюдаю проблемы

Вообще от слова deprecated надо избавляться как можно быстрее.

bogachev-pa commented 9 years ago
  1. Вместо actionsSets и action стоит создавать меню через menuContribution, menu и command, как это делается в основном рдо: https://github.com/aurusov/rdo-xtext/blob/release/current/ru.bmstu.rk9.rao.ui/plugin.xml#L498, а в качестве обработчика вместо Game5Action нужно завести хэндлер команды.
  2. Почему wontfix?
aurusov commented 9 years ago

Потому что это не бага, но исправить надо. Как бы ты назвал ?

bogachev-pa commented 9 years ago

Я использую для этого лэйбл refactoring обычно. Вообще, я думал wontfix означает, что проблема существует, но ее решение настолько трудоемко, что она фикситься не будет.

aurusov commented 9 years ago

Блин, использовал wontfix в смысле wantfix :) Рефакторинг - может быть, но уж больно он для многих случаев подходит, т.е. значение размывается. Тогда это бага, чтобы не плодить статусы.

LeKaitoW commented 9 years ago

Вместо actionsSets и action стоит создавать меню через menuContribution, menu и command, как это делается в основном рдо: https://github.com/aurusov/rdo-xtext/blob/release/current/ru.bmstu.rk9.rao.ui/plugin.xml#L498, а в качестве обработчика вместо Game5Action нужно завести хэндлер команды.

Сделано именно так, ворнингов нет.

aurusov commented 9 years ago

Давай добавим visibleWhen, как Паша написал.

LeKaitoW commented 9 years ago

Добавила условие. Пулл реквест ставить?

aurusov commented 9 years ago

Ага

LeKaitoW commented 9 years ago

Задача решена: LeKaitoW/rdo-game5#20