Скрипт автозакрытия предупреждений и вопросов без моего спроса стал все закрывать

tormozit Закрыто

Версия 5.15.7719.28093 от 18 февраля 2021 г.
После сегодняшнего обновления все вопросы и предупреждения стали сами закрываться без моего согласия на это.
Clip_209672.png

Комментарии

bolsun
#1, 19 февраля 2021 09:08

Какие например?


tormozit
#2, ред. 19 февраля 2021 09:09

Так картинка же есть - перезапуск отлаживаемого приложения


bolsun
#3, 19 февраля 2021 09:11

(2) tormozit, что я должен понять из нее?


Cmapnep
#4, 19 февраля 2021 09:12

А мне вопрос в первый раз задал.
Но вопрос общий для всех настроек, а их в текущем варианте 5 штук.
Я решил подстраховаться и выключил опасные.
Получилось так:
...
Настройки = Новый Структура();
Настройки.Вставить("ОшибкиВМодуле", Ложь);
Настройки.Вставить("ПриложениеЗапущено", Ложь);
Настройки.Вставить("ПрекратитьОтладку", Ложь);
Настройки.Вставить("КонфигурацияОтличается", Истина);
Настройки.Вставить("СвязьРазорвана", Истина);


tormozit
#5, ред. 19 февраля 2021 09:14

Картинка показывает положение курсора мыши непосредственно перед нажатием левой кнопки мыши в состоянии отладки клиентского приложения.


bolsun
#6, 19 февраля 2021 09:14

(5) tormozit, это я понял. Какое окно закрывает?
В процессе ее нажатия может быть выдано несколько разных предупреждений. Про то что приложение запущено?


tormozit
#7, ред. 19 февраля 2021 09:15

"все вопросы и предупреждения" которые тут появляются. Заметить их было сложно, т.к. очень быстро нажимались кнопки скриптом.


bolsun
#8, 19 февраля 2021 09:15

(7) tormozit, тогда нужно настроить необходимые предупреждения, как выше написал Cmapnep


tormozit
#9, 19 февраля 2021 09:16

Не, кодить я не хочу). Подожду исправления.


bolsun
#10, 19 февраля 2021 09:17

(9) tormozit, это не ошибка, исправления поэтому не будет.
Это файл настроек скрипта.


bolsun
#11, 19 февраля 2021 09:20

В настройках скрипта ./settings/Zakryvashka/ можно выбрать предупреждения, которые необходимо автоматически закрывать.


tormozit
#12, 19 февраля 2021 09:20

На предложение закрывать предупреждение я только один раз отвеил "да" - на предыдущей версии турбоконфа для окошка "Связь ... с потеряна", но предупреждение тогда не закрылось и я сразу же написал по этому поводу багрепорт. В новой версии ты добавил поддержку этого предупреждения судя по списку изменений. Но у меня теперь все предупреждения стали закрываться.


Cmapnep
#13, 19 февраля 2021 09:21

Скрипт все же опасный, как ни крути)
Может стоит сделать значения по умолчанию = ЛОЖЬ
А окно выводить при каждом новом случае и при этом менять в файле настроек для этого и только этого пункта значение?
Вроде это возможно


tormozit
#14, 19 февраля 2021 09:22

Ну тогда есть баг с записью в этот файл настоек. Я не соглашался закрывать все предупреждения.


tormozit
#15, ред. 19 февраля 2021 09:24

Папки ./settings/Zakryvashka/ у меня вообще нет.
Удалил сам скрипт. Теперь нормально работает.


tormozit
#16, 19 февраля 2021 09:26

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


Cmapnep
#17, 19 февраля 2021 09:26

(14) tormozit, а там нет записи в файл настроек сейчас - они только вручную редактируются
А ответ на вопрос влияет только глобально на использование скрипта - у тебя как раз так и получилось
Тут логическая проблема - нельзя все настроки включать вместе - мало ли сколько их там, на что повлияют и какие еще настройки появятся в будущем - все будут разрешены 😦


bolsun
#18, 19 февраля 2021 09:27

(14) tormozit, окно выдается один раз для всех предупреждений. В тексте вопроса было сказано, что закрываться будут все окна, которые есть в списке настроек. Поэтому соглашался ))

TurboConf_5_15_autoclose_query.jpg


bolsun
#19, ред. 19 февраля 2021 09:28

(16) tormozit, т.е. у тебя нет папки settings ? в корне программы?


tormozit
#20, 19 февраля 2021 09:30

Я ВНИМАТЕЛЬНО прочитал это предупреждение. Про "все" в нем не написано. Написано про "подобные". Я это понял как ID окна/вопроса. У меня даже и мысли не было, что можно было делать одну настройку для всех вопросов. ЭТО ОЧЕНЬ ОПАСНО. Вроде бы обсуждали это в чате


bolsun
#21, 19 февраля 2021 09:30

Мне проще тогда совсем убрать скрипт из поставки. Кому нужно скачает с форума и заменит пару Истина на Ложь или наоборот.


tormozit
#22, 19 февраля 2021 09:30

(19) bolsun, папка settings есть


bolsun
#23, 19 февраля 2021 09:31

(20) tormozit, про ВСЕ не написано. Написано про то, что закрываться будут те, которые указаны в настройках скрипта.


tormozit
#24, 19 февраля 2021 09:32

(21) bolsun, да, тогда лучше убрать скрипт вообще. Если он не позволяет удобно задавать реакцию отдельно для каждого типа предупрежедения.


bolsun
#25, 19 февраля 2021 09:35

(22) tormozit, а папка Zakryvashka есть в ней? Мне просто интересно, как скрипт мог работать без файла настроек.


Cmapnep
#26, 19 февраля 2021 09:37

(21) bolsun, предлагаю все же остановиться на компромиссном варианте:

  1. В настройках по умолчанию оставить все значения = ЛОЖЬ
  2. Логику появления вопроса оставить такой же, но переделать вопрос в информационное окно, в котором писать, что есть скрипт "Закрывашка", который умеет автоматически закрывать подобные окна и что его нужно настроить под себя
    Итого минимум переделок и все довольны)


bolsun
#27, 19 февраля 2021 09:38

(26) Cmapnep, да в целом можно и на каждый тип предупреждения выводить запрос. Там добавить нужно несколько строк в скрипте (на первый взгляд).


bolsun
#28, 19 февраля 2021 09:41

Еще получается нужно в первый раз вывести вопрос, вообще "Вы хотите включить скрипт?".
Затем выводить на каждый тип отдельное предупреждение.


tormozit
#29, 19 февраля 2021 09:42

Еще важный момент. Если скрипт позволяет себя просто включить, то исходя из его опасности, должен быть и сопоставимо простой способ его выключить. Т.е. прямо в предложении включить этот скрипт должно быть четкое обозначение, где и как его отключить и это "как" не должно включить в себя кодинг)


tormozit
#30, 19 февраля 2021 09:43

(25) bolsun, я же написал что папки ./settings/Zakryvashka/ у меня нет.


bolsun
#31, ред. 19 февраля 2021 09:48

(29) tormozit, так же как и сейчас сделано, флаг включается/отключается в настройках мышкой. Это под силу, наверное, среднестатистическому разработчику.


bolsun
#32, 19 февраля 2021 09:46

Делать отдельную кнопку во всех меню не буду ))


bolsun
#33, 19 февраля 2021 09:47

Тут еще проблема, если будет 10 типов уведомлений. Не надоест каждый раз соглашаться?


tormozit
#34, 19 февраля 2021 09:47

  1. Выше мне предлагался кодинг.
  2. В вопросе про включение скрипта сейчас не указан четко способ его включить. Нужно хотя бы указать название скрипта и что это вообще скрипт.


bolsun
#35, 19 февраля 2021 09:52

Еще проблема, если пользователь согласился закрывать какой то тип предупреждений, а потом передумал. Как он будет это менять? Так бы он уже знал, где что отключить/включить в файле настроек. В общем сложность задачи, похоже растет в прогрессии ))


tormozit
#36, ред. 19 февраля 2021 09:57

По-хорошему конечно надо делать в настройках список типов окошек с пометками или даже с вариантами реакции

  • предложить автозакрытие
  • закрывать
  • игнорировать
    Первый вариант реакции будет полезен, если юзер случайно ответил "Да/Нет" и не помнит где это было (что за диалог) и хочет снова принять решение, когда увидит его в следующий раз.


Cmapnep
#37, 19 февраля 2021 10:24

Ну вот и пришли к тому, что нужен комбайн целый)
А потом выяснится, что для кого-то все равно реализовано не удобно
Дешевле все же отдать настройки на откуп пользователю
Просто предупредить при первом появлении окна, которое скрипт может закрывать, что есть такая возможность и для этого нужно редактировать файл пользовательских настроек скрипта
Кто-то настроит, кто-то забьет, но оба будут контролировать ситуацию, что в данном случае самое главное!


tormozit
#38, 19 февраля 2021 10:40

Извиняюсь. Торопился. Папка ./settings/Zakryvashka/ у меня есть. Ранее я ошибочно смотрел в папке scripts.


bolsun
#39, ред. 19 февраля 2021 11:10

И так, предлагаю определиться с минимальным и достаточным набором функционала.

  1. При первом срабатывании скрипта задать вопрос об общем включении скрипта "Автозакрытие окон". Сообщить где можно в дальнейшем вкл/выкл скрипт
  2. Если пользователь согласился, показывать отдельный вопрос для каждого типа предупреждений.
  3. Сделать функцию для сброса настроек скрипта, разместить ее в ТурбоМеню/Скрипты

Решить нужно ли показывать всплывающее уведомление для всех типов предупреждений, а не только для ошибок при проверке синтаксиса.


bolsun
#40, ред. 19 февраля 2021 11:18

Вот пробный вариант. Скачать в папку scripts. Нажать в конфигураторе Ctrl+Shift+R для перезагрузки скриптов.
Реализованы пункты 1 и 2

Zakryvashka.os


bolsun
#41, 19 февраля 2021 11:43

Просьба написать, кто попробовал.


Cmapnep
#42, 19 февраля 2021 12:48

Не могу добиться появления вопроса
Да и судя по всему скрипт не запускается...
Я обновил скрипт и перезагрузил chrgbns? и перезапускал турбоконф тоже - главный вопрос не задается и окна не закрываются
Возможно дело в сохраненной настройке глобальной?
Ее как-то можно сбросить?


bolsun
#43, 19 февраля 2021 12:54

(42) Cmapnep, похоже понял в чем дело.
Обновил скрипт Zakryvashka.os


Cmapnep
#44, 19 февраля 2021 13:03

(43) bolsun, да, заработало. Поведение такое, как описал выше.
Есть вопрос - теперь файл пользовательских настроек не используется?
Настройки сохраняются только в UserValues.tc?


bolsun
#45, 19 февраля 2021 13:05

(44) Cmapnep, не используются больше. А нужны они теперь?


Cmapnep
#46, 19 февраля 2021 13:09

(45) bolsun, я спросил с целью понять как мне изменить свое решение, если я передумал
С отдельным файлом все было очевидно, да и описано в стандартном файле настроек
А сейчас не совсем очевидно стало
Если я правильно понимаю, то при внесении изменений руками в этот файл нужно перезапускать TC
Или нужно закрыть TC, потом внести изменения в файл и запустить TC снова?
В общем было бы неплохо чтобы где-то это было описано, например в файле со стандартными настройками


bolsun
#47, 19 февраля 2021 13:12

(46) Cmapnep, иногда текстовый файлик с настройками, удобнее и проще в использовании ))
значит нужна форма настроек, а еще пользователь должен знать как ее вызвать...


bolsun
#48, 19 февраля 2021 13:44

(46) Cmapnep, такую форму можно добавить
2021-02-19_154223.png


Cmapnep
#49, 19 февраля 2021 13:50

(48) bolsun, выглядит отлично и в настройке удобно
Сбросить все вообще супер)


bolsun
#50, 19 февраля 2021 14:05

(49) Cmapnep, обвновил скрипт. В ТурбоМеню/Скрипты появится "Настройка автозакрытия окон". Просьба проверить.
Zakryvashka.os


Cmapnep
#51, 19 февраля 2021 14:34

(50) bolsun, кажется все идеально, спасибо!


bolsun
#52, 25 февраля 2021 14:54

bolsun изменил статус на Завершено


Для вставки изображения или файла, перетащите его в поле редактора или вставьте файл из буфера