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

Роли #33

Open
elagin opened this issue Jul 25, 2015 · 5 comments
Open

Роли #33

elagin opened this issue Jul 25, 2015 · 5 comments
Assignees
Labels
Milestone

Comments

@elagin
Copy link
Owner

elagin commented Jul 25, 2015

По первых не правильно работает getSession().isRO(): Даже когда Session.role = "standart" она возвращает false.
По вторых если быстро нажать "зеленый плюс", то при role = "Не установлена" isRO опять возвращает false, тут вроде как правильно, но получается, что безправный пользователь может создать точку.

@elagin elagin added the bug label Jul 25, 2015
@elagin elagin added this to the first blood milestone Jul 25, 2015
@rjhdby
Copy link
Collaborator

rjhdby commented Jul 26, 2015

Это философский момент.
Роль ReadOnly уникальна. Все остальные роли обычно используются в разрешающем контексте. Типа если стандарт и выше, то можно. ReadOnly же используется в контексте запрещающем, если RO, то нельзя.
Поэтому так и завел. isRO возвращает true, если таки действительно RO, а все остальные true если равно и выше.

@elagin
Copy link
Owner Author

elagin commented Jul 26, 2015

%) Сделай, пожалуйста, что бы в случае "ридонли" и "не известно' не могли точки создавать

@elagin
Copy link
Owner Author

elagin commented Jul 28, 2015

При роли "стадтарт" isRO возвращает истину.

@elagin
Copy link
Owner Author

elagin commented Jul 29, 2015

Временно можно пользоваться так: 3e22e75

@elagin elagin modified the milestones: 2.0, first blood Jul 29, 2015
@elagin
Copy link
Owner Author

elagin commented Jul 29, 2015

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants