[WEBCLIENT-342] switch response status check from "OK" to 200 to handle lab value pop up in http/2 protocols Created: 26/Jan/21 Updated: 15/Jun/22 Resolved: 25/May/22 |
|
| Status: | Closed |
| Project: | i2b2 Web Client |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 1.7.13 |
| Type: | Bug | Priority: | Major |
| Reporter: | Reeta Metta | Assignee: | Reeta Metta |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Participant/s: |
| Description |
|
There are several places in the CRC cell surrounding the setvalue javascript that reference the statusText which were failing on successful responses because the statusText was empty on servers returning http/2 protocols.
Change theresponse status check from "OK" to 200 instead for lab value pop up handling Github commit: 2 2 changes: 1 addition & 1 deletion js-i2b2/cells/CRC/ModLabValues/CRC_view_ENUM.js @@ -32,7 +32,7 @@ i2b2.CRC.view.ENUM = { 32 32 var url = "js-i2b2/cells/CRC/ModLabValues/CRC_view_ENUM.html"; 33 33 var response = new Ajax.Request(url, {method: 'get', asynchronous: false}); 34 34 console.dir(response); 35 - if (response.transport.statusText=="OK") { 35 + if (response.transport.status == 200) { 36 36 thisHTML = response.transport.responseText; 37 37 } else { 38 38 alert('A problem was encounter while loading the html for the value type!'); |