Author: markt
Date: Wed Mar  3 23:42:33 2010
New Revision: 918779

URL: http://svn.apache.org/viewvc?rev=918779&view=rev
Log:
Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=48686
Fix NPE in host delete

Modified:
    tomcat/tc5.5.x/trunk/STATUS.txt
    
tomcat/tc5.5.x/trunk/container/webapps/admin/WEB-INF/classes/org/apache/webapp/admin/host/DeleteHostAction.java
    tomcat/tc5.5.x/trunk/container/webapps/docs/changelog.xml

Modified: tomcat/tc5.5.x/trunk/STATUS.txt
URL: 
http://svn.apache.org/viewvc/tomcat/tc5.5.x/trunk/STATUS.txt?rev=918779&r1=918778&r2=918779&view=diff
==============================================================================
--- tomcat/tc5.5.x/trunk/STATUS.txt (original)
+++ tomcat/tc5.5.x/trunk/STATUS.txt Wed Mar  3 23:42:33 2010
@@ -113,12 +113,6 @@
   +1: kkolinko, rjung
   -1: 
 
-* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=48686
-  Fix NPE in host delete
-  https://issues.apache.org/bugzilla/attachment.cgi?id=24966
-  +1: markt, kkolinko, rjung
-  -1: 
-
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=48582
   Avoid NPE on background compile
   http://svn.apache.org/viewvc?rev=910471&view=rev

Modified: 
tomcat/tc5.5.x/trunk/container/webapps/admin/WEB-INF/classes/org/apache/webapp/admin/host/DeleteHostAction.java
URL: 
http://svn.apache.org/viewvc/tomcat/tc5.5.x/trunk/container/webapps/admin/WEB-INF/classes/org/apache/webapp/admin/host/DeleteHostAction.java?rev=918779&r1=918778&r2=918779&view=diff
==============================================================================
--- 
tomcat/tc5.5.x/trunk/container/webapps/admin/WEB-INF/classes/org/apache/webapp/admin/host/DeleteHostAction.java
 (original)
+++ 
tomcat/tc5.5.x/trunk/container/webapps/admin/WEB-INF/classes/org/apache/webapp/admin/host/DeleteHostAction.java
 Wed Mar  3 23:42:33 2010
@@ -113,7 +113,7 @@
         // this host cannot be deleted from the admin tool
         try {
             adminHost = Lists.getAdminAppHost(
-                                  mBServer, "domain" ,request);
+                                  mBServer, domain ,request);
         } catch (Exception e) {
             String message =
                 resources.getMessage(locale, "error.hostName.bad",

Modified: tomcat/tc5.5.x/trunk/container/webapps/docs/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/tc5.5.x/trunk/container/webapps/docs/changelog.xml?rev=918779&r1=918778&r2=918779&view=diff
==============================================================================
--- tomcat/tc5.5.x/trunk/container/webapps/docs/changelog.xml (original)
+++ tomcat/tc5.5.x/trunk/container/webapps/docs/changelog.xml Wed Mar  3 
23:42:33 2010
@@ -278,6 +278,10 @@
         in the documentation web application to specify how the defaults for 
the
         various trust store attributes are determined. (markt)
       </add>
+      <fix>
+        <bug>48686</bug>: Fix deleting a host via the Administration web
+        application rather than failign with a HTTP 500 response. (markt)
+      </fix>
       <add>
         Make changelog.xml be directly rendered as HTML by certain browsers.
         (kkolinko)



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to