Skip to content
  • P
    Projects
  • G
    Groups
  • S
    Snippets
  • Help

CIRCLE / cloud

  • This project
    • Loading...
  • Sign in
Go to a project
  • Project
  • Repository
  • Issues 94
  • Merge Requests 10
  • Pipelines
  • Wiki
  • Snippets
  • Members
  • Activity
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Merged
Merge request !79 opened May 27, 2014 by Bach Dániel@bachdaniel 
  • Report abuse
Report abuse

Feature Create User

tests 🆗 ~~👷 👷 ~~

  • Discussion 2
  • Commits 4
  • Changes
{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • Őry Máté
    @orymate started a discussion on the diff May 28, 2014
    circle/dashboard/views.py
    2583 template_name = 'dashboard/user-create.html'
    2584
    2585 def get_group(self, group_pk):
    2586 self.group = get_object_or_404(Group, pk=group_pk)
    2587 if not self.group.profile.has_level(self.request.user, 'owner'):
    2588 raise PermissionDenied()
    2589
    2590 def get(self, *args, **kwargs):
    2591 if not self.request.user.has_perm('auth.add_user'):
    2592 raise PermissionDenied()
    2593 self.get_group(kwargs.pop('group_pk'))
    2594 return super(UserCreationView, self).get(*args, **kwargs)
    2595
    2596 def post(self, *args, **kwargs):
    2597 if not self.request.user.has_perm('auth.add_user'):
    2598 raise PermissionDenied()
    • Őry Máté @orymate commented May 28, 2014
      Owner

      I'd prefer http://django-braces.readthedocs.org/en/latest/access.html#permissionrequiredmixin fixed (fcf7744c)

      Edited May 28, 2014
      ~~I'd prefer http://django-braces.readthedocs.org/en/latest/access.html#permissionrequiredmixin~~ fixed (fcf7744c)
    Please register or sign in to reply
  • Őry Máté @orymate commented May 28, 2014
    Owner

    👍

    :+1:
  • Bach Dániel
    @bachdaniel started a discussion on commit fcf7744c May 28, 2014
    • Bach Dániel @bachdaniel

      mentioned in merge request !79 (merged)

      May 28, 2014

      mentioned in merge request !79 (merged)

      mentioned in merge request !79
      Toggle commit list
    Please register or sign in to reply
  • Write
  • Preview
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or sign in to comment
Bach Dániel
Assignee
Bach Dániel @bachdaniel
Assign to
Nice to have
Milestone
Nice to have
Assign milestone
Time tracking
0
Labels
None
Assign labels
  • View labels
2
2 participants
Reference: circle/cloud!79