Skip to content

Commit

Permalink
Remove FOSUser Controllers
Browse files Browse the repository at this point in the history
  • Loading branch information
jordisala1991 committed Mar 25, 2017
1 parent c029525 commit cf8dd4d
Show file tree
Hide file tree
Showing 10 changed files with 0 additions and 421 deletions.
87 changes: 0 additions & 87 deletions Controller/ChangePasswordFOSUser2Controller.php

This file was deleted.

149 changes: 0 additions & 149 deletions Controller/ProfileFOSUser2Controller.php

This file was deleted.

18 changes: 0 additions & 18 deletions Controller/RegistrationFOSUser2Controller.php

This file was deleted.

21 changes: 0 additions & 21 deletions Controller/ResettingFOSUser2Controller.php

This file was deleted.

41 changes: 0 additions & 41 deletions Controller/SecurityFOSUser2Controller.php

This file was deleted.

9 changes: 0 additions & 9 deletions Resources/config/routing/sonata_change_password_2.xml

This file was deleted.

21 changes: 0 additions & 21 deletions Resources/config/routing/sonata_profile_2.xml

This file was deleted.

27 changes: 0 additions & 27 deletions Resources/config/routing/sonata_registration_2.xml

This file was deleted.

4 comments on commit cf8dd4d

@Prometee
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why this deletions ? Do we have now to redefine the entire controllers and routes to work with SonataUserBundle ?

@jordisala1991
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I removed routes that only override those defined by fos_user. You don't have to redefine anything if you want to simply have a user login on your admin.

This code was used to define another login system for your frontend part. But you can already do that using FOSUser default routes, so those are just pointless.

@Prometee
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, I thought that this branch will be merge to the 4.x one one day but it seems that it will live a life alone.

@jordisala1991
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#869

This branch will be merged, but not with a regular merge, but with that PR

Please sign in to comment.