You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@oozie.apache.org by rk...@apache.org on 2013/06/14 22:50:26 UTC

svn commit: r1493239 - in /oozie/trunk: release-log.txt webapp/src/main/webapp/oozie-console.js

Author: rkanter
Date: Fri Jun 14 20:50:25 2013
New Revision: 1493239

URL: http://svn.apache.org/r1493239
Log:
OOZIE-1365 The hive action popup in the web UI is broken when externalChildIDs is empty string (michalisk via rkanter)

Modified:
    oozie/trunk/release-log.txt
    oozie/trunk/webapp/src/main/webapp/oozie-console.js

Modified: oozie/trunk/release-log.txt
URL: http://svn.apache.org/viewvc/oozie/trunk/release-log.txt?rev=1493239&r1=1493238&r2=1493239&view=diff
==============================================================================
--- oozie/trunk/release-log.txt (original)
+++ oozie/trunk/release-log.txt Fri Jun 14 20:50:25 2013
@@ -1,5 +1,6 @@
 -- Oozie 4.1.0 release (trunk - unreleased)
 
+OOZIE-1365 The hive action popup in the web UI is broken when externalChildIDs is empty string (michalisk via rkanter)
 OOZIE-1412 Webapp contains all sharedlib dependencies after launcher refactor (rohini)
 OOZIE-1414 Configuring Oozie for HTTPS still allows HTTP connections to all resources (rkanter)
 OOZIE-1410 V2 servlets are missing from ssl-web.xml (rkanter, rohini via rkanter)

Modified: oozie/trunk/webapp/src/main/webapp/oozie-console.js
URL: http://svn.apache.org/viewvc/oozie/trunk/webapp/src/main/webapp/oozie-console.js?rev=1493239&r1=1493238&r2=1493239&view=diff
==============================================================================
--- oozie/trunk/webapp/src/main/webapp/oozie-console.js (original)
+++ oozie/trunk/webapp/src/main/webapp/oozie-console.js Fri Jun 14 20:50:25 2013
@@ -565,7 +565,7 @@ function jobDetailsPopup(response, reque
 	function populateUrlUnit(actionStatus, urlUnit) {
 		var consoleUrl = actionStatus["consoleUrl"];
         var externalChildIDs = actionStatus["externalChildIDs"];
-		if(undefined !== consoleUrl && null !== consoleUrl && undefined !== externalChildIDs && null !== externalChildIDs) {
+		if(!consoleUrl && !externalChildIDs) {
 	        var urlPrefix = consoleUrl.trim().split(/_/)[0];
             //externalChildIds is a comma-separated string of each child job ID.
             //Create URL list by appending jobID portion after stripping "job"