Две настройки попали в lexical analyzer и не видны в релизе

Issue #310 closed
Oleg Sychev repo owner created an issue

Originally reported on Google Code with ID 310

По логике им место там, но по сути они должны быть доступны при любом одном из двух
включенных анализаторов - или lexical или sequence. По идее они наверное должны быть
доступны всегда, так что можно и в общий блок настроек вынести.

Нужен быстрый воркараунд чтобы сделать версию 2.6.1
Блок и вопрос в релизной версии находятся в центральном репозитории проекта.

Reported by oasychev on 2014-11-28 10:41:17

Comments (6)

  1. Former user Account Deleted
    Вытолкнул исправление в своем клоне. Но не уверен, что это хорошая идея, выливать мои
    изменения. Может лучше склонировать ваш и сделать в нем ветку? Или в принципе и мой
    можно?
    

    Reported by mamontov.dp on 2014-11-30 18:07:43

  2. Oleg Sychev reporter
    По идее если мы хотим интегрировать код от Бирюковой к новому релизу - а уже давно надо
    было - то эти изменения в нем не нужны. Их нужно сделать в релизнутой копии вопроса,
    которую можно склонировать с основного репозитория этого проекта. И сделать там версию
    типа 2.6.1 
    Можно просто переслать мне исправленные файлы, я подготовлю и залью эту версию...
    

    Reported by oasychev on 2014-11-30 18:11:16

  3. Oleg Sychev reporter
    Прошу протестировать итоговый пакет - создать/отредактировать вопрос, убедиться что
    новые настройки сохраняются в БД
    

    Reported by oasychev on 2014-12-02 17:33:11

    <hr> * Attachment: correctwriting_261.zip

  4. Former user Account Deleted
    Протестировал на edu.vstu.org, вроде проблем нет, поля появились и сохраняются.
    

    Reported by mamontov.dp on 2014-12-04 04:48:40

  5. Log in to comment