Skip to content

Commit b7443d9

Browse files
committed
Merge branch 'master' of github.com:dcmjs-org/dicomweb-client into feat/enhancers
2 parents 4b9b26a + 4799d1e commit b7443d9

File tree

2 files changed

+40
-12
lines changed

2 files changed

+40
-12
lines changed

package-lock.json

Lines changed: 6 additions & 6 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/api.js

Lines changed: 34 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,7 @@ class DICOMwebClient {
5959
* @param {String} options.password - Password
6060
* @param {Object} options.headers - HTTP headers
6161
* @param {Array.<RequestHook>} options.requestHooks - Request hooks.
62+
* @param {Object} options.verbose - print to console request warnings and errors, default true
6263
*/
6364
constructor(options) {
6465
this.baseURL = options.url;
@@ -106,6 +107,27 @@ class DICOMwebClient {
106107

107108
// Optional error interceptor callback to handle any failed request.
108109
this.errorInterceptor = options.errorInterceptor || function() {};
110+
111+
// Verbose - print to console request warnings and errors, default true
112+
this.verbose = options.verbose === false ? false : true;
113+
}
114+
115+
/**
116+
* Sets verbose flag.
117+
*
118+
* @param {Boolean} verbose
119+
*/
120+
setVerbose(verbose) {
121+
this.verbose = verbose
122+
}
123+
124+
/**
125+
* Gets verbose flag.
126+
*
127+
* @return {Boolean} verbose
128+
*/
129+
getVerbose() {
130+
return this.verbose;
109131
}
110132

111133
static _parseQueryParameters(params = {}) {
@@ -165,24 +187,30 @@ class DICOMwebClient {
165187
};
166188

167189
// Handle response message
168-
request.onreadystatechange = function onreadystatechange() {
190+
request.onreadystatechange = () => {
169191
if (request.readyState === 4) {
170192
if (request.status === 200) {
171193
resolve(request.response);
172194
} else if (request.status === 202) {
173-
console.warn("some resources already existed: ", request);
195+
if (this.verbose) {
196+
console.warn("some resources already existed: ", request);
197+
}
174198
resolve(request.response);
175199
} else if (request.status === 204) {
176-
console.warn("empty response for request: ", request);
200+
if (this.verbose) {
201+
console.warn("empty response for request: ", request);
202+
}
177203
resolve([]);
178204
} else {
179-
console.error("request failed: ", request);
180205
const error = new Error("request failed");
181206
error.request = request;
182207
error.response = request.response;
183208
error.status = request.status;
184-
console.error(error);
185-
console.error(error.response);
209+
if (this.verbose) {
210+
console.error("request failed: ", request);
211+
console.error(error);
212+
console.error(error.response);
213+
}
186214

187215
errorInterceptor(error);
188216

0 commit comments

Comments
 (0)