alex-rufous commented on issue #42: Qpid-8374: [Broker-J][ACL] Allow case 
insensitive mapping of group members to groups in existing GroupProvider
URL: https://github.com/apache/qpid-broker-j/pull/42#issuecomment-559799524
 
 
   > Hi Alex @alex-rufous,
   > I have one more question about CloudFoundaryDashboardManagement.
   > 
   > **code(cloudfoundarydashboardmanagement/show.js):**
   > var table = query(".serviceToManagementGroupMappingTable", 
this.containerNode)[0];
   > console.log("Check 1: " + table);
   > table.innerHTML = tableContent;
   > console.log("Check 2: " + tableContent);
   > console.log("Check 3: " + table);
   > 
   > **Opening GroupProvider panel for CloudFoundary I can see console output 
below: (at 1st moment table is undefined). Should I leave it as it is?**
   > Check 1: undefined show.js:48:25
   > TypeError: table is undefined show.js:49:17
   > Check 1: [object HTMLTableElement] show.js:48:25
   > Check 2: 1: new2: new_2 show.js:50:25
   > Check 3: [object HTMLTableElement]
   > 
   > Thank you.
   > Best Regards,
   > Stanislav
   
   Hi Stanislav,
   It is an existing defect in cloudfoundary dashboard management UI.
   It is caused by asynchronous loading and parsing of HTML template for the 
tab.
   The method #update() can be called before the loading of template is 
completed. As result, the query cannot find the node with class 
'serviceToManagementGroupMappingTable'. 
   It could be fixed by checking the query output. The  innerHTML should be set 
only  when query returns node object.
   
   Kind Regards,
   Alex

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to