diff options
author | Holger Weiss <holger@zedat.fu-berlin.de> | 2015-11-09 18:41:31 +0100 |
---|---|---|
committer | Holger Weiss <holger@zedat.fu-berlin.de> | 2015-11-09 18:41:31 +0100 |
commit | 9c80cb2b26f950a14a0a1312868fdf97e5c442af (patch) | |
tree | f795f92aa8234b1fbc2c6b97d13d662c07b8e532 /src/mod_http_upload.erl | |
parent | Use xml_stream:reset() when possible (diff) |
mod_http_upload: Don't let identify/1 return error
Let identify/1 return 'pass' when it failed to identify the file type,
as this doesn't (necessarily) indicate an error condition. This also
makes it consistent with the return value of convert/2.
Diffstat (limited to 'src/mod_http_upload.erl')
-rw-r--r-- | src/mod_http_upload.erl | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/mod_http_upload.erl b/src/mod_http_upload.erl index 72642665..3e154d1c 100644 --- a/src/mod_http_upload.erl +++ b/src/mod_http_upload.erl @@ -776,7 +776,7 @@ store_file(Path, Data, FileMode, DirMode, GetPrefix, LocalPath, Thumbnail) -> <<"text/xml; charset=utf-8">>}], xml:element_to_binary(ThumbEl)} end; - {error, _} -> + pass -> ok end; ok -> @@ -873,7 +873,7 @@ code_to_message(_Code) -> <<"">>. %%-------------------------------------------------------------------- %% Image manipulation stuff %%-------------------------------------------------------------------- --spec identify(binary()) -> {ok, media_info()} | {error, binary()}. +-spec identify(binary()) -> {ok, media_info()} | pass. identify(Path) -> Cmd = lists:flatten(io_lib:fwrite("identify -format \"ok %m %h %w\" ~s", @@ -887,7 +887,7 @@ identify(Path) -> width = list_to_integer(W)}}; _ -> ?DEBUG("failed to identify type of ~s: ~s", [Path, Res]), - {error, list_to_binary(Res)} + pass end. -spec convert(binary(), media_info()) -> {ok, binary()} | pass. @@ -929,7 +929,7 @@ thumb_el(Path, URI) -> {<<"uri">>, URI}, {<<"height">>, jlib:integer_to_binary(H)}, {<<"width">>, jlib:integer_to_binary(W)}]}; - {error, _} -> + pass -> #xmlel{name = <<"thumbnail">>, attrs = [{<<"xmlns">>, ?NS_THUMBS_1}, {<<"uri">>, URI}, |