/*
 * HeadsUp Agile
 * Copyright 2014 Heads Up Development Ltd.
 *
 * This program is free software: you can redistribute it and/or modify
 * it under the terms of the GNU Affero General Public License as
 * published by the Free Software Foundation, either version 3 of the
 * License, or (at your option) any later version.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU Affero General Public License for more details.
 *
 * You should have received a copy of the GNU Affero General Public License
 * along with this program.  If not, see <http://www.gnu.org/licenses/>.
 */
package org.headsupdev.agile.web.wicket;

import org.apache.wicket.util.convert.converters.AbstractConverter;
import org.headsupdev.agile.storage.issues.Duration;

import java.util.Locale;

/**
 * Created by Gordon Edwards on 02/07/2014.
 */
public class DurationConverter
        extends AbstractConverter
{
    @Override
    protected Class<?> getTargetType()
    {
        return Duration.class;
    }

    @Override
    public Object convertToObject( String value, Locale locale )
    {
        Duration duration = null;
        try
        {
            duration = Duration.fromString( value );
        }
        catch ( Exception e )
        {
            throw newConversionException( "Invalid", value, locale );
        }
        return duration;
    }
}

@@ -893,16 +893,63 @@
 uration;
+%0Aimport org.headsupdev.support.java.StringUtil;
 %0A%0Aimport
@@ -1016,16 +1016,33 @@
 7/2014.%0A
+ *%0A * @since 2.1%0A
  */%0Apubl
@@ -1298,34 +1298,89 @@
     
-Duration duration =
+if ( StringUtil.isEmpty( value ) )%0A        %7B%0A            return
  null;%0A
+        %7D%0A%0A
     
@@ -1413,18 +1413,14 @@
     
-duration =
+return
  Dur
@@ -1580,33 +1580,8 @@
   %7D%0A
-        return duration;%0A
     

Support a null duration worked - setRequired can be used if we need non-null - fixes issue:89