Bug #58517
Integer arent allowed as object identifier in routes
Status: | Resolved | Start date: | 2014-05-05 | |
---|---|---|---|---|
Priority: | Should have | Due date: | ||
Assigned To: | Bastian Waidelich | % Done: | 100% |
|
Category: | MVC - Routing | |||
Target version: | - | |||
PHP Version: | Complexity: | |||
Has patch: | No | Affected Flow version: | Git master |
Description
I want to have an integer as persistence object identifier, so I implemented it this way:
/** * @var int * @ORM\Id * @ORM\GeneratedValue * @Flow\Identity */ protected $id;
But flow raised an exception while it tries to resolve the route for the show action for my object. I think the following part of the core is the problem, isnt it?
I attached my patch.
Related issues
Associated revisions
[BUGFIX] Support numeric identifiers in dynamic route parts
Currently the dynamic route part handler provided by Flow (namely
``DynamicRoutePart`` and ``IdentityRoutePart``) only support objects
with string identifiers.
With this change also numeric identifiers are supported, for example
for entities with a numeric auto_increment identifier.
Change-Id: I57da5820c38d08494937f011439c00b0fd6851a7
Fixes: #48887
Fixes: #58517
Releases: master, 2.3, 2.2
[BUGFIX] Support numeric identifiers in dynamic route parts
Currently the dynamic route part handler provided by Flow (namely
``DynamicRoutePart`` and ``IdentityRoutePart``) only support objects
with string identifiers.
With this change also numeric identifiers are supported, for example
for entities with a numeric auto_increment identifier.
Change-Id: I57da5820c38d08494937f011439c00b0fd6851a7
Fixes: #48887
Fixes: #58517
Releases: master, 2.3, 2.2
[BUGFIX] Support numeric identifiers in dynamic route parts
Currently the dynamic route part handler provided by Flow (namely
``DynamicRoutePart`` and ``IdentityRoutePart``) only support objects
with string identifiers.
With this change also numeric identifiers are supported, for example
for entities with a numeric auto_increment identifier.
Change-Id: I57da5820c38d08494937f011439c00b0fd6851a7
Fixes: #48887
Fixes: #58517
Releases: master, 2.3, 2.2
History
#1 Updated by Bastian Waidelich about 1 year ago
- Project changed from TYPO3 Flow Base Distribution to TYPO3.Flow
#2 Updated by Bastian Waidelich about 1 year ago
- Category set to MVC - Routing
- Status changed from New to Accepted
- Assigned To set to Bastian Waidelich
- Has patch set to No
Thanks for reporting
#3 Updated by Gerrit Code Review about 1 year ago
- Status changed from Accepted to Under Review
Patch set 1 for branch master of project Packages/TYPO3.Flow has been pushed to the review server.
It is available at https://review.typo3.org/30116
#4 Updated by Gerrit Code Review about 1 year ago
Patch set 2 for branch master of project Packages/TYPO3.Flow has been pushed to the review server.
It is available at https://review.typo3.org/30116
#5 Updated by Gerrit Code Review 6 months ago
Patch set 3 for branch master of project Packages/TYPO3.Flow has been pushed to the review server.
It is available at http://review.typo3.org/30116
#6 Updated by Gerrit Code Review 6 months ago
Patch set 1 for branch 2.3 of project Packages/TYPO3.Flow has been pushed to the review server.
It is available at http://review.typo3.org/36997
#7 Updated by Gerrit Code Review 6 months ago
Patch set 1 for branch 2.2 of project Packages/TYPO3.Flow has been pushed to the review server.
It is available at http://review.typo3.org/36998
#8 Updated by Bastian Waidelich 6 months ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 4e698ac3d86195f87863a2d17759871b4b9990f2.
#9 Updated by Gerrit Code Review 4 months ago
- Status changed from Resolved to Under Review
Patch set 2 for branch 2.2 of project Packages/TYPO3.Flow has been pushed to the review server.
It is available at http://review.typo3.org/36998
#10 Updated by Gerrit Code Review 3 months ago
Patch set 3 for branch 2.2 of project Packages/TYPO3.Flow has been pushed to the review server.
It is available at http://review.typo3.org/36998
#11 Updated by Bastian Waidelich 3 months ago
- Status changed from Under Review to Resolved
Applied in changeset 8687bfb8c34fe4a50febff3f968152decd98aaa3.