Merge tag 'v3.0.1' into fiksgatami-dev

1 files changed, 7 insertions, 1 deletions

diff --git a/perllib/FixMyStreet/Roles/ConfirmValidation.pm b/perllib/FixMyStreet/Roles/ConfirmValidation.pm
index 776230287..27592b33c 100644
--- a/perllib/FixMyStreet/Roles/ConfirmValidation.pm
+++ b/perllib/FixMyStreet/Roles/ConfirmValidation.pm
@@ -17,6 +17,8 @@ Confirm field lengths.
 
 has max_report_length => ( is => 'ro', default => 2000 );
 
+has max_title_length => ( is => 'ro', default => 0 );
+
 sub report_validation {
     my ($self, $report, $errors) = @_;
 
@@ -24,10 +26,14 @@ sub report_validation {
         $errors->{name} = sprintf( _('Names are limited to %d characters in length.'), 50 );
     }
 
-    if ( length( $report->user->phone ) > 20 ) {
+    if ( $report->user->phone && length( $report->user->phone ) > 20 ) {
         $errors->{phone} = sprintf( _('Phone numbers are limited to %s characters in length.'), 20 );
     }
 
+    if ( $self->max_title_length > 0 && length( $report->title ) > $self->max_title_length ) {
+        $errors->{title} = sprintf( _('Summaries are limited to %d characters in length. Please shorten your summary'), 50 );
+    }
+
     if ( length( $report->detail ) > $self->max_report_length ) {
         $errors->{detail} = sprintf( _('Reports are limited to %s characters in length. Please shorten your report'), $self->max_report_length );
     }