From 56bf69ccc4cb0ec2809ee85f187c7a79b490fb27 Mon Sep 17 00:00:00 2001 From: Brett Leslie Porter Date: Mon, 3 Oct 2005 14:41:51 +0000 Subject: [PATCH] PR: MNG-644 remove deprecated default-value handling git-svn-id: https://svn.apache.org/repos/asf/maven/components/trunk@293360 13f79535-47bb-0310-9956-ffa450edef68 --- .../maven/plugin/DefaultPluginManager.java | 33 ------------------- 1 file changed, 33 deletions(-) diff --git a/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java b/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java index 60e94f6cc3..fada74777a 100644 --- a/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java +++ b/maven-core/src/main/java/org/apache/maven/plugin/DefaultPluginManager.java @@ -782,39 +782,6 @@ public class DefaultPluginManager throw new PluginConfigurationException( "Bad expression", e ); } - if ( fieldValue == null && goal.getComponentConfigurator() == null ) - { - try - { - // TODO: would like to remove in beta-1 as configurator should always be given, but it is - // currently causing problems with published plugins. - Field field = findPluginField( plugin.getClass(), parameter.getName() ); - boolean accessible = field.isAccessible(); - if ( !accessible ) - { - field.setAccessible( true ); - } - fieldValue = field.get( plugin ); - if ( !accessible ) - { - field.setAccessible( false ); - } - if ( fieldValue != null ) - { - getLogger().warn( "DEPRECATED: using default-value to set the default value of field '" + - parameter.getName() + "'" ); - } - } - catch ( NoSuchFieldException e ) - { - throw new PluginConfigurationException( "Unable to find field to check default value", e ); - } - catch ( IllegalAccessException e ) - { - throw new PluginConfigurationException( "Unable to read field to check default value", e ); - } - } - // only mark as invalid if there are no child nodes if ( fieldValue == null && ( value == null || value.getChildCount() == 0 ) ) {