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

Расширенный поиск - правки по 2 ISSUES #144

Merged
merged 5 commits into from
Aug 21, 2021
Merged

Расширенный поиск - правки по 2 ISSUES #144

merged 5 commits into from
Aug 21, 2021

Conversation

7OH
Copy link
Contributor

@7OH 7OH commented Feb 7, 2021

Правки по #126 и #140

реализация #

По #140 добавлены галки для включения или отключения отбора по метаданным.
А так же само поле, которое работает почти также как в скрипте "Навигатор по метаданным"

По #126 добавлена галка "Вертикальное окно" в настройки основного окна.
Сделать одной формой не вышло, так как платформа не даёт менять вид прикрепления окна на ходу.
Сделано двумя формами, которые и открываются в зависимости от галки.
В работе есть один нюанс - менять настройку на ходу можно, но перестают работать кнопки на панели задач.
Может @orefkov сможет что-то позже придумать, но пока на этом месте стоит try\catch

// По непонятной причине при установке свойства Action
// возникает исключение "Объект не поддерживает это свойство или метод",
// но свойство все равно устанавливается, поэтому для обхода проблемы
// исключение проглатываем.
try {
buttons[i].Action = action;
} catch(e){ /*Message(e.description);*/}

Сделал выбор основной формы при первой инициализации по настройке - так работает.
В общем определились как больше нравится и не меняем.
Или после изменения - перезайти.

Проверено на релизе 8.3.17.1851

@infostart-hub/snegopat - предлагаю прокомментировать и проверить

Правки по #126 и #140
@artbear
Copy link
Contributor

artbear commented Feb 8, 2021

@7OH странные результаты
image

@artbear artbear added the script label Feb 8, 2021
@artbear artbear self-assigned this Feb 8, 2021
@artbear artbear changed the title Правки по 2 ISSUES Расширенный поиск - правки по 2 ISSUES Feb 8, 2021
Copy link
Contributor

@artbear artbear left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

есть непонятности
ответишь?

addins/extSearchReplace.js Outdated Show resolved Hide resolved
addins/extSearchReplace.js Outdated Show resolved Hide resolved
addins/extSearchReplace.js Outdated Show resolved Hide resolved
Автоустановка\снятие галки отбора по именам при изменении строки отбора.
Причесал код скрипта
addins/extSearchReplace.js Outdated Show resolved Hide resolved
Поправил сравнение.
Вынес сравнение ещё на уровень выше.
Замена IN на обычный цикл с итератором
addins/extSearchReplace.js Outdated Show resolved Hide resolved
addins/extSearchReplace.js Outdated Show resolved Hide resolved
@7OH
Copy link
Contributor Author

7OH commented Feb 13, 2021

Пробовал поиск на нашей ЕРП.
Текст в задаче был (последний же элемент в дереве).
Без фильтра - 7 минут.
С фильтром - 10 секунд.

@artbear
Copy link
Contributor

artbear commented Feb 23, 2021

@kuntashov можешь проверить поведение? у меня были странные проблемы с этим скриптом (
сейчас вроде работает, но осадочек остался )

@artbear
Copy link
Contributor

artbear commented Feb 23, 2021

для больших конфигураций добавленный нами фильтр очень полезен.

хочется побыстрее внедрить в работу.

@kuntashov может быть, интерфейсные фишки подскажешь, чтобы улучшить

@artbear
Copy link
Contributor

artbear commented Aug 21, 2021

ПР из ядерного Снегопата infostart-hub/snegopat_modul#21

@artbear artbear merged commit dff83cb into infostart-hub:master Aug 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants