Commitc3d4046

authored and

committed

Still implement deprecated username method

This updates the makers forsymfony/symfony#41493

1 parent4edd87e commitc3d4046Copy full SHA for c3d4046

File tree

+20

-0

lines changed

src
- Resources/skeleton/security
  - UserProvider.tpl.php
- Security
  - UserClassBuilder.php

+20

-0

lines changed

Lines changed: 10 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -28,6 +28,16 @@ public function <?= $uses_user_identifier ? 'loadUserByIdentifier($identifier)'`
`28`	`28`	`throw new \Exception('TODO: fill in<?=$uses_user_identifier ?'loadUserByIdentifier()' :'loadUserByUsername()'?> inside '.__FILE__);`
`29`	`29`	`}`
`30`	`30`
	`31`	`+<?phpif ($uses_user_identifier) :?>`
	`32`	`+ /**`
	`33`	`+ * @deprecated since Symfony 5.3, loadUserByIdentifier() is used instead`
	`34`	`+ */`
	`35`	`+ public function loadUserByUsername($username): UserInterface`
	`36`	`+ {`
	`37`	`+ return $this->loadUserByIdentifier($username);`
	`38`	`+ }`
	`39`	`+<?phpendif?>`
	`40`	`+`
`31`	`41`	`/**`
`32`	`42`	`* Refreshes the user after being reloaded from the session.`
`33`	`43`	`*`

Lines changed: 10 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -103,6 +103,16 @@ private function addGetUsername(ClassSourceManipulator $manipulator, UserClassCo`
`103`	`103`	`// Check if we're using Symfony 5.3+ - UserInterface::getUsername() was replaced with UserInterface::getUserIdentifier()`
`104`	`104`	`if (class_exists(InMemoryUser::class)) {`
`105`	`105`	`$getterIdentifierName ='getUserIdentifier';`
	`106`	`+`
	`107`	`+// also add the deprecated getUsername method`
	`108`	`+$manipulator->addAccessorMethod(`
	`109`	`+$userClassConfig->getIdentityPropertyName(),`
	`110`	`+'getUsername',`
	`111`	`+'string',`
	`112`	`+false,`
	`113`	`+ ['@deprecated since Symfony 5.3, use getUserIdentifier instead'],`
	`114`	`+true`
	`115`	`+ );`
`106`	`116`	`}`
`107`	`117`
`108`	`118`	`// define getUsername (if it was defined above, this will override)`

Comments

(0)