TYPO3 Flow Base DistributionPackagesApplications

Bug #34641

Comment's message should be > 255 chars

Added by Adrian Föder over 3 years ago. Updated about 3 years ago.

Status:Resolved Start date:2012-03-07
Priority:Should have Due date:
Assigned To:Adrian Föder % Done:

100%
Category:-
Target version:-

Description

Currently the property message is not specially annotated and therefore limited to a max length of 255; which should, for a message, clearly be more.

Associated revisions

Revision 48d284f6
Added by Adrian Föder over 3 years ago

[BUGFIX] Allows message length to be > 255 chars

This patch corrects the allowed message length to be
over 255 chars.

Change-Id: Ie0b91fd7ec0d88e967dd1cddd5c05d485f51975f
Fixes: #34641

History

#1 Updated by Bastian Waidelich over 3 years ago

Good catch. The missing annotation is:

1@ORM\Column(type="text")

#2 Updated by Bastian Waidelich over 3 years ago

  • Status changed from New to Accepted

#3 Updated by Adrian Föder over 3 years ago

  • Assigned To set to Adrian Föder

#4 Updated by Adrian Föder over 3 years ago

Bastian, could you please, taking this opportunity, explain me how to handle that migration file stuff? This change would result in a new Migration set, right? How do I create such one?

Thanks!
adrian

#5 Updated by Bastian Waidelich over 3 years ago

Adrian Föder wrote:

Bastian, could you please, taking this opportunity, explain me how to handle that migration
file stuff? This change would result in a new Migration set, right? How do I create such one?

Hi,
it's mostly described here: http://flow3.typo3.org/documentation/guide/partiii/persistence.html#schema-management

  • Adjust the model until everything is correct
  • Execute ./flow3 doctrine:migrationgenerate
  • Move the created file from Data/Migrations to Your.Package/Migrations/Mysql
  • Do manual adjustments to the migration, if necessary

#6 Updated by Adrian Föder over 3 years ago

  • Tracker changed from Task to Bug

#7 Updated by Gerrit Code Review over 3 years ago

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/9645

#8 Updated by Adrian Föder over 3 years ago

  • Status changed from Accepted to Under Review
  • % Done changed from 0 to 60

#9 Updated by Gerrit Code Review over 3 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/9645

#10 Updated by Gerrit Code Review over 3 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/9645

#11 Updated by Adrian Föder about 3 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 60 to 100

Also available in: Atom PDF