Задача: разрешить пользователям проводить документы только внутри разрешенных им складов.
Частично данная задача успешно решается при помощи RLS и соответствующих ролей. Но! не полностью. Во всяком случае в конфигурации 1С Розница 2.2 нет возможности ограничить пользователя (без изменения конфигураци)… ну, например, реализовывать товар с "чужого" склада/Магазина, делать перемещение товаров на "чужие" склады (ограничения работают только для Ордеров???).
Решение: используем относительно новую фичу от 1С — подписка на события. А именно на событие "Запись". Данные, по которым будем проверять, можно пользователю записывать данный документ или нет, будем брать из специально созданного справочника.
В котором перечислена группа, к которой принадлежит пользователь, вид документа, по которому его ограничиваем, и список складов, с которыми ему позволено это документ использовать. Далее создадим подписку на событие:
И собственно саму обработку события
Функция ПолучитьГруппуПользователя()
Запрос = Новый Запрос;
Запрос.Текст = "ВЫБРАТЬ
| ГруппыПользователей.Наименование КАК Наименование
|ИЗ
| Справочник.ГруппыПользователей КАК ГруппыПользователей
|ГДЕ
| ГруппыПользователей.Состав.Пользователь.Ссылка = &Ссылка";
Запрос.УстановитьПараметр("Ссылка", ПараметрыСеанса.ТекущийПользователь);
РезультатЗапроса = Запрос.Выполнить();
ВыборкаДетальныеЗаписи = РезультатЗапроса.Выбрать();
группа="";
Пока ВыборкаДетальныеЗаписи.Следующий() Цикл
группа=ВыборкаДетальныеЗаписи.Наименование;
конеццикла;
возврат группа;
КонецФункции
Функция ПроверкаВсяческихПрав(Источник, Отказ) Экспорт
отказываем=истина;
естьправила=ложь;
гп= ПолучитьГруппуПользователя();
естьОтправитель=ложь;
естьПолучатель=ложь;
отказываемОтправитель=истина;
отказываемПолучатель=истина;
// проверяем соответствие складам "Откуда"
Запрос = Новый Запрос;
Запрос.Текст =
"ВЫБРАТЬ
| ДополнительныеНастройкиПравГрибовСкладыОткуда.Склад.Ссылка КАК СкладСсылка
|ИЗ
| Справочник.ДополнительныеНастройкиПравГрибов.РазрешенныеСклады КАК ДополнительныеНастройкиПравГрибовСкладыОткуда
|ГДЕ
| ДополнительныеНастройкиПравГрибовСкладыОткуда.Ссылка.ГруппаПользователей.Наименование = &ИмяГруппы
| И ДополнительныеНастройкиПравГрибовСкладыОткуда.Ссылка.ВидДокумента.Наименование = &ИмяДокумента";
Запрос.УстановитьПараметр("ИмяГруппы", гп);
Запрос.УстановитьПараметр("ИмяДокумента", Источник.Ссылка.Метаданные().Имя);
РезультатЗапроса = Запрос.Выполнить();
ВыборкаДетальныеЗаписи = РезультатЗапроса.Выбрать();
Пока ВыборкаДетальныеЗаписи.Следующий() Цикл
естьправила=Истина;
если Источник.Ссылка.Метаданные().Реквизиты.Найти("Склад")<>Неопределено тогда
если источник.Склад.Ссылка=ВыборкаДетальныеЗаписи.СкладСсылка тогда отказываем=Ложь;
конецесли;
конецесли;
если Источник.Ссылка.Метаданные().Реквизиты.Найти("СкладОтправитель")<>Неопределено тогда
естьОтправитель=Истина;
если источник.СкладОтправитель.Ссылка=ВыборкаДетальныеЗаписи.СкладСсылка тогда
отказываем=Ложь;
отказываемОтправитель=Ложь;
конецесли;
конецесли;
если Источник.Метаданные().Реквизиты.Найти("СкладПолучатель")<>Неопределено тогда
естьПолучатель=Истина;
если источник.Ссылка.СкладПолучатель.Ссылка=ВыборкаДетальныеЗаписи.СкладСсылка тогда
отказываем=Ложь;
отказываемОтправитель=Ложь;
конецесли;
конецесли;
конеццикла;
если естьправила=Ложь тогда
Отказ=ложь;
иначе
//отрабатываем вариант что два склада отправитель и получатель!
если (естьОтправитель=Истина и естьПолучатель=Истина) тогда
если отказываемОтправитель=Ложь и отказываемПолучатель=Ложь тогда
отказ=ложь;
иначе
отказ=Истина;
конецесли;
иначе
отказ=отказываем;
конецесли
конецесли;
если отказ=Истина тогда
сообщить("- вы не можете записать данный документ с данными складами - не хватает прав! Обратитесь к Администратору.");
конецесли;
КонецФункции
P.S. Это не "готовое решение", а пример как можно сделать. Есть и другие способы.
(0)
чего-чего?
насколько новую? 🙂
почему наименование, а не сама группа?
Можно было бы сделать все одним запросом…
А если групп будет несколько?
Склад.Ссылка Т___Т Не надо так…
А если до этого сработали типовые проверки и Отказ = Истина, а вы его так смело Отказ = Ложь…
И все можно было сделать в запросе…
эммммммм…я ее еще в году так 2011 использовал …то есть минимум 6 лет это относительно новое??
А что за грибы ?
Не совсем понимаю для чего там подписка на событие?
Молчу об РЛС.
— если пользователь вызвал просто перепровести из списка документа (что тут такого просто перепроведется документ не вижу никакого криминала);
— хотите ограничить права таким способом (пожалуйста), а что мешает в форме просто перед записью объекта проверку сделать
(5)
— хотите ограничить права таким способом (пожалуйста), а что мешает в форме просто перед записью объекта проверку сде
Религия! видимо автор хотел «модно» не снимать с поддержки … но почему тогда не расширение которое как раз для этого и служит … (конечно оно не такое новое как подписка на события)
(4) см. фамилию автора
Ну, хочется человеку оставить свой след 🙂
а вообще, за такой код
хочется по рукам надавать. Вот как надо:
найди 3 отличия!
1000 и 1 способ выстрелить себе в ногу. Поставил минус — хочу в меру сил уменьшить шанс встретить поделия учеников такого мастера.
Кто считает, что слишком жестко — посмотрите другие публикации автора. Я не поленился перед выставлением минуса и коммента.
(7)
Только при условии что в переменной будет тип булево. Если там будет какой то другой тип то будет ошибка.
Вариант «Если ЕстьПравила = Ложь Тогда» будет более устойчивым к ошибкам.
(9)
Код проанализируйте. Там не может быть другой тип
(9) Если в переменной тип не булево, то это еще один косяк программиста. То что среда разработки позволяет писать говнокод еще не повод действительно писать его
(10)
Речь была не о конкретном коде. А об утверждении что за такой подход надо руки отрывать.
(11)
Конечно косяк программиста. А чей ещё?
Но почему то люди пишут говнокод. И другим с ним приходится работать. В реальном мире живём… а не в сферическом конфигураторе в вакууме.
(13)
И в чем смысл этого высказывания? А еще есть люди которые едят фекалии, как прокомментируете? Мол в реальном мире живем, пусть едят в общих столовых рядом с остальными? =)
Если некое отрицательное явление имеет сравнительно широкое распространение в отдельных нишах, то это не повод это явление возводить в норму.
(13) Ну, тогда в каждой строке кода писать Попытку?
Чтобы уже точно защититься
(14)
Перечитайте 10 раз если не понятно с первого. Ваше сравнение с фикалиями тут не уместно. Просто потому что открываю типовую розницу.
Открываю поиск по конфигурации и ввожу в поиск «Истина Тогда» и получаю 903 результата….
А кто то возводит?
(15)
Если ты точно знаешь что твоя функция возвращает тип булево то нет. Как и написано в «системе стандартов». А если ты точно не уверен?
(16) А кто пишет этот код? Стажеры малооплачиваемые.
Помню видел в каком-то регл.отчете комментарий примерно такой — «Не знаю зачем, на всякий случай»
(15)
Для начала не трактовать написанное в стандартах разработки буквально.
(19)
Может систему стандартов то же они писали?
(20) Вы часто упоминаете эту систему, но как-то в общем. Скажите конкретно, в чем я неправ в данном конкретном случае.
(21)
А я где то утверждал что вы не правы?
Я вроде как написал что в коде может быть сравнение и это нормальная практика. Аргументировал вроде… скриншот приложил….
Вот пример из типовой (соседние строка кода)
Кто такое мог написать?! Сколько нужно иметь извилин, чтобы не писать условие?
(23) Может быть индусы?
(24) Суровый индусский код
(23)
Что вас так смущает в этом коде? И как должно быть написано правильно?
(26) Вы что, издеваетесь? Или хочется пофлудить?
(27)Исключительно в познавательных целях. Просто интересно в чём по вашему мнению ошибка?
(26) Для чего нужна проверка условия
? 🙂
(28)
(29) Ага. Но товарищу, похоже, это трудно понять
(28)
Как правило, переменная Отказ уже объявлена
Не хочу обидеть человека, но за такой код я бы руки отрывал. Ну хоть Ctrl + A, Alt+Shift+F нажимали.
Ну просто все позабавились …
Да ладно. Все так делают, когда времени мало, а базу можно курочить.
Но когда время есть и хочется сделать все правильно — делают через RLS.
А такие поделки обычно не выкладывают, ими стыдятся
(35) вот прямо в точку. Либо студент и первое его перо.
я такой код встречал когда человек с 7.7 переходил на 8. ещё не приловчился походу
Код не по КодСтайлу. Запросы ужасны. Идея дрянь. Забаньте это пожалуйста.
Внедряли RLS в типовых, ничего нового не кодили, просто расставляли галочки в типовом РЛС в КА 1.1. Тот еще шлак получился, когда база тормозить стала. Нужно уметь курить этот RLS. В некоторых ситуациях, проще и надежнее, по старинке, как в 7.7 прописать, в стандартные процедуры. Зато не тормозит, и не нужно серверный ключик клиенту покупать.