1237603 - Sample code "convert the binary hash data to a hex string"

Status: RESOLVED FIXED

(Developer Documentation Graveyard :: General, defect)

Description

[Alice0775 White]

Bug 1220564 was landed, 
I think the sample code should be revised.
https://developer.mozilla.org/en-US/Add-ons/SDK/Tutorials/Creating_reusable_modules

Comment 1

[Tooru Fujisawa [:arai]]

Updated the page and localized pages, with code copied from patch in bug 1220564.

https://developer.mozilla.org/en-US/Add-ons/SDK/Tutorials/Creating_reusable_modules
https://developer.mozilla.org/pt-BR/docs/Mozilla/Add-ons/SDK/Tutorials/Creating_reusable_modules

https://developer.mozilla.org/en-US/docs/Mozilla/Tech/XPCOM/Reference/Interface/nsICryptoHash
https://developer.mozilla.org/ko/docs/nsICryptoHash
https://developer.mozilla.org/ja/docs/XPCOM_Interface_Reference/nsICryptoHash

Comment 2

[Tooru Fujisawa [:arai]]

Feel free to reopen when the change is not sufficient.

Comment 3

[Alice0775 White]

Thanks!

Comment 4

[Sören Hentzschel]

The documentation has been updated to:

var s = Array.from(hash, (c, i) => toHexChar(hash.charCodeAt(i))).join("");

But it's wrong, toHexChar() is undefined. The function name is toHexString().

Comment 5

[Tooru Fujisawa [:arai]]

(In reply to Sören Hentzschel from comment #4)
> The documentation has been updated to:
> 
> var s = Array.from(hash, (c, i) => toHexChar(hash.charCodeAt(i))).join("");
> 
> But it's wrong, toHexChar() is undefined. The function name is toHexString().

Thank you!
Fixed.