Skip to content

Commit f7ee2bd

Browse files
committed
fix(image): uncaught error in promise with image handling
1 parent 8e83a61 commit f7ee2bd

File tree

2 files changed

+17
-3
lines changed

2 files changed

+17
-3
lines changed

tns-core-modules/http/http-request/http-request.ios.ts

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -69,6 +69,11 @@ function ensureImageSource() {
6969
export function request(options: http.HttpRequestOptions): Promise<http.HttpResponse> {
7070
return new Promise<http.HttpResponse>((resolve, reject) => {
7171

72+
if (!options.url) {
73+
reject('Request url was empty.');
74+
return;
75+
}
76+
7277
try {
7378
var network = domainDebugger.getNetwork();
7479
var debugRequest = network && network.create();

tns-core-modules/http/http.ts

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -31,9 +31,18 @@ export function getJSON<T>(arg: any): Promise<T> {
3131
}
3232

3333
export function getImage(arg: any): Promise<ImageSource> {
34-
return httpRequest
35-
.request(typeof arg === "string" ? { url: arg, method: "GET" } : arg)
36-
.then(response => response.content.toImage());
34+
return new Promise<any>((resolve, reject) => {
35+
httpRequest.request(typeof arg === "string" ? { url: arg, method: "GET" } : arg)
36+
.then(r => {
37+
try {
38+
resolve(r.content.toImage());
39+
} catch (err) {
40+
reject(err);
41+
}
42+
}, err => {
43+
reject(err);
44+
});
45+
});
3746
}
3847

3948
export function getFile(arg: any, destinationFilePath?: string): Promise<any> {

0 commit comments

Comments
 (0)