На жаль, позитивного співробітництва не склалося, і я так і не зрозумів претензій до себе, але відповім на них:
1 .Було 2 завдання: а) Виведення списку поточних користувачів з ролями, їх ролі в адмін.панелі(АП).б) Можливість адміністратору призначити користувача менеджером в АП.Олександр зробив сторінку з усіма користувачами, на якій можна призначити управлінську роль.Але це не підходить для завдання "А" - на сторінці не можна побачити тільки тих, хто з управлінською роллю.На прикладі: дозвольмо, потрібно подивитися всіх менеджерів.У системі 1000 користувачів, і щоб зібрати список менеджерів - потрібно переглядати всіх 1000 користувачів (50 сторінок х 20 користувачів).Я, просто, доброзичливо вказував на проблему, на яку отримав роздратування і грубості, мол: «у чому священний сенс роботи, як будуть назначатися ролі тим, у кого немає ролі?« »Так, поточне рішення не вирішило поставлені завдання, але адже рішення не я ж писав, а в ТЗ описані вимоги.Але для того, щоб уникнути переписання рішення - я запропонував користувачам з управлінською роллю просто виводити в вершину.У чому моя провина – не розумію:
1) У ТЗ вимоги описані чітко.2) Я доброзичливо і без грубостей просто вказував, що не вирішується одна з завдань.3) Потім запропонував альтернативне рішення, яке дозволило не переписувати існуюче.2 .Заявки до коду проекту: «написана ахінея».Але не було конкретики, і проблеми спостерігаються у виконавця...
1) На прохання уточнити в чому проблема - відповідь не була отримана.Затвердження задувало urlManager, але він досить «стандартний»: [https://i.imgur.com/r12W2OH.png]
2) У завданнях більшість взаємодій йде з Yii'шним RBAC.3) Приклади коду проекту були застосовані.Я пропонував до початку співпраці перевірити репозиторий.Але в коді самого Олександра є проблеми.Правда, в звичайних випадках, я на це не звертаю уваги, бо: це все робиться досить просто, і відмінний спосіб навчитися для програміста/ревівера.- DRY: для пошуку користувачів з роллю був копійований метод, з додаванням строки orderBy.Статичний аналітик IDE це підсвітлив: [https://i.imgur.com/wLahUe2.png] (але це було виправлено після зауважень)
- У сервіси було затягнуто Singeltone, хоча там використовується контейнер залежності, і AuthManager конфігурується, і дрібниці від стат.аналізатора IDE відображаються: https://i.imgur.com/d1w8dfs.png
- З ТЗ: "У користувача може бути тільки одна "менеджерська роль"(!)одночасно ." - поточний код Олександра відв'язує не тільки управлінську роль, а й будь-яку іншу: https://i.imgur.com/1Q1Wbit.png
- Для завантаження JS для відправлення команди ролі - використовується костил: https://i.imgur.com/OJ8a783.png
Хоча є рішення з коробки: \yii\web\View::registerJsFile.- Імена комітів Git - неінформативні: "admin roles : in progress", "task : in progress", "small changes"
Шкода, що так сталося.Але я вважаю, що створив максимальні умови для комфортного вирішення завдання.
1 .Було 2 завдання: а) Виведення списку поточних користувачів з ролями, їх ролі в адмін.панелі(АП).б) Можливість адміністратору призначити користувача менеджером в АП.Олександр зробив сторінку з усіма користувачами, на якій можна призначити управлінську роль.Але це не підходить для завдання "А" - на сторінці не можна побачити тільки тих, хто з управлінською роллю.На прикладі: дозвольмо, потрібно подивитися всіх менеджерів.У системі 1000 користувачів, і щоб зібрати список менеджерів - потрібно переглядати всіх 1000 користувачів (50 сторінок х 20 користувачів).Я, просто, доброзичливо вказував на проблему, на яку отримав роздратування і грубості, мол: «у чому священний сенс роботи, як будуть назначатися ролі тим, у кого немає ролі?« »Так, поточне рішення не вирішило поставлені завдання, але адже рішення не я ж писав, а в ТЗ описані вимоги.Але для того, щоб уникнути переписання рішення - я запропонував користувачам з управлінською роллю просто виводити в вершину.У чому моя провина – не розумію:
1) У ТЗ вимоги описані чітко.2) Я доброзичливо і без грубостей просто вказував, що не вирішується одна з завдань.3) Потім запропонував альтернативне рішення, яке дозволило не переписувати існуюче.2 .Заявки до коду проекту: «написана ахінея».Але не було конкретики, і проблеми спостерігаються у виконавця...
1) На прохання уточнити в чому проблема - відповідь не була отримана.Затвердження задувало urlManager, але він досить «стандартний»: [https://i.imgur.com/r12W2OH.png]
2) У завданнях більшість взаємодій йде з Yii'шним RBAC.3) Приклади коду проекту були застосовані.Я пропонував до початку співпраці перевірити репозиторий.Але в коді самого Олександра є проблеми.Правда, в звичайних випадках, я на це не звертаю уваги, бо: це все робиться досить просто, і відмінний спосіб навчитися для програміста/ревівера.- DRY: для пошуку користувачів з роллю був копійований метод, з додаванням строки orderBy.Статичний аналітик IDE це підсвітлив: [https://i.imgur.com/wLahUe2.png] (але це було виправлено після зауважень)
- У сервіси було затягнуто Singeltone, хоча там використовується контейнер залежності, і AuthManager конфігурується, і дрібниці від стат.аналізатора IDE відображаються: https://i.imgur.com/d1w8dfs.png
- З ТЗ: "У користувача може бути тільки одна "менеджерська роль"(!)одночасно ." - поточний код Олександра відв'язує не тільки управлінську роль, а й будь-яку іншу: https://i.imgur.com/1Q1Wbit.png
- Для завантаження JS для відправлення команди ролі - використовується костил: https://i.imgur.com/OJ8a783.png
Хоча є рішення з коробки: \yii\web\View::registerJsFile.- Імена комітів Git - неінформативні: "admin roles : in progress", "task : in progress", "small changes"
Шкода, що так сталося.Але я вважаю, що створив максимальні умови для комфортного вирішення завдання.
