[jboss-cvs] JBossAS SVN: r67086 - projects/metadata/trunk/src/main/java/org/jboss/metadata/web/jboss.

jboss-cvs-commits at lists.jboss.org jboss-cvs-commits at lists.jboss.org
Wed Nov 14 13:28:19 EST 2007


Author: bstansberry at jboss.com
Date: 2007-11-14 13:28:19 -0500 (Wed, 14 Nov 2007)
New Revision: 67086

Modified:
   projects/metadata/trunk/src/main/java/org/jboss/metadata/web/jboss/PassivationConfig.java
   projects/metadata/trunk/src/main/java/org/jboss/metadata/web/jboss/ReplicationConfig.java
Log:
[JBAS-4960] Remove the extra properties used during conversion

Modified: projects/metadata/trunk/src/main/java/org/jboss/metadata/web/jboss/PassivationConfig.java
===================================================================
--- projects/metadata/trunk/src/main/java/org/jboss/metadata/web/jboss/PassivationConfig.java	2007-11-14 18:18:01 UTC (rev 67085)
+++ projects/metadata/trunk/src/main/java/org/jboss/metadata/web/jboss/PassivationConfig.java	2007-11-14 18:28:19 UTC (rev 67086)
@@ -26,11 +26,6 @@
 /**
  * Represents a <passivation-config> element of the jboss-web.xml 
  * deployment descriptor 
- *
- * FIXME There are overloaded versions of most of the properties here that are of
- * type String.  This is just part of a dance to convert the AS to the real types
- * without having to risk breaking the AS build. I'll migrate the API over a couple
- * snapshot releases of jboss-metadata.jar.
  * 
  * @author Brian Stansberry
  * @version <tt>$Revision$</tt>
@@ -40,8 +35,6 @@
    /** The serialVersionUID */
    private static final long serialVersionUID = 1;
 
-   private static final String IGNORE = "-1";
-   
    /**
     * If true then the session passivation is enabled for this web app.
     * If false then the session passivation is disabled for this web app. 
@@ -87,37 +80,6 @@
       return useSessionPassivation;
    }
 
-   /** FIXME Remove after next snapshot publication */
-   public Integer getPassivationMaxIdleTimeInteger()
-   {
-      return passivationMaxIdleTime;
-   }
-
-   /** FIXME Remove after next snapshot publication */
-   public void setPassivationMaxIdleTimeInteger(Integer passivationMaxIdleTime)
-   {
-      this.passivationMaxIdleTime = passivationMaxIdleTime;
-   }
-
-   /** FIXME Remove after next snapshot publication */
-   public Integer getPassivationMinIdleTimeInteger()
-   {
-      return passivationMinIdleTime;
-   }
-
-   /** FIXME Remove after next snapshot publication */
-   public void setPassivationMinIdleTimeInteger(Integer passivationMinIdleTime)
-   {
-      this.passivationMinIdleTime = passivationMinIdleTime;
-   }
-
-   /** FIXME Remove after next snapshot publication */
-   public Boolean getUseSessionPassivationBoolean()
-   {
-      return useSessionPassivation;
-   }
-
-   /** FIXME Remove after next snapshot publication */
    public void setUseSessionPassivation(Boolean useSessionPassivation)
    {
       this.useSessionPassivation = useSessionPassivation;

Modified: projects/metadata/trunk/src/main/java/org/jboss/metadata/web/jboss/ReplicationConfig.java
===================================================================
--- projects/metadata/trunk/src/main/java/org/jboss/metadata/web/jboss/ReplicationConfig.java	2007-11-14 18:18:01 UTC (rev 67085)
+++ projects/metadata/trunk/src/main/java/org/jboss/metadata/web/jboss/ReplicationConfig.java	2007-11-14 18:28:19 UTC (rev 67086)
@@ -25,11 +25,6 @@
 
 /**
  * Represents a <replication-config> element of the jboss-web.xml deployment descriptor 
- *
- * FIXME There are overloaded versions of most of the properties here that are of
- * type String.  This is just part of a dance to convert the AS to the real types
- * without having to risk breaking the AS build. I'll migrate the API over a couple
- * snapshot releases of jboss-metadata.jar.
  * 
  * @author <a href="mailto:bdecoste at jboss.com">William DeCoste</a>
  * @version <tt>$Revision$</tt>
@@ -95,18 +90,6 @@
       this.snapshotInterval = snapshotInterval;
    }
 
-   /** FIXME Remove after next snapshot publication */
-   public Integer getSnapshotIntervalInteger()
-   {
-      return snapshotInterval == null ? 0 : snapshotInterval.intValue();
-   }
-
-   /** FIXME Remove after next snapshot publication */
-   public void setSnapshotIntervalInteger(Integer snapshotInterval)
-   {
-      this.snapshotInterval = snapshotInterval;
-   }
-
    public SnapshotMode getSnapshotMode()
    {
       return snapshotMode;
@@ -117,18 +100,6 @@
       this.snapshotMode = snapshotMode;
    }
 
-   /** FIXME Remove after next snapshot publication */
-   public SnapshotMode getSnapshotModeEnum()
-   {
-      return snapshotMode;
-   }
-
-   /** FIXME Remove after next snapshot publication */
-   public void setSnapshotModeEnum(SnapshotMode snapshotMode)
-   {
-      this.snapshotMode = snapshotMode;
-   }
-
    public Boolean getUseJK()
    {
       return useJK;
@@ -139,18 +110,6 @@
       this.useJK = useJK;
    }
 
-   /** FIXME Remove after next snapshot publication */
-   public Boolean getUseJKBoolean()
-   {
-      return useJK;
-   }
-
-   /** FIXME Remove after next snapshot publication */
-   public void setUseJKBoolean(Boolean useJK)
-   {
-      this.useJK = useJK;
-   }
-
    public String toString()
    {
       StringBuffer sb = new StringBuffer(100);




More information about the jboss-cvs-commits mailing list