dbaq / angular-emoji-filter-hd

An AngularJS filter for replacing emoji codes with high-definition emoticons (Retina friendly).
Other
39 stars 16 forks source link

version 0.0.8 have a major bug #11

Closed thatkookooguy closed 8 years ago

thatkookooguy commented 8 years ago

I'm trying to replace emojis inside a message that contains some html.

on version 0.0.6 it worked perfectly every time. After updating to 0.0.8, It started getting broken

I send the following text (parsed for presentation):

<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
    <style type="text/css" style="display:none;">
    <!-- P {margin-top:0;margin-bottom:0;} -->
    </style>
  </head>
  <body dir="rtl">
    <div id="divtagdefaultwrapper" style="font-size:12pt;color:#000000;background-color:#FFFFFF;font-family:Calibri,Arial,Helvetica,sans-serif;">
      <p>This is great! :wink:</p>
    </div>
  </body>
</html>

and I get the following with 0.0.6:

<html> <head> <meta http-equiv="Content-Type" content="text/html; charset=utf-8"> <style type="text/css" style="display:none;"> <!-- P {margin-top:0;margin-bottom:0;} --> </style> </head> <body dir="rtl"> <div id="divtagdefaultwrapper" style="font-size:12pt;color:#000000;background-color:#FFFFFF;font-family:Calibri,Arial,Helvetica,sans-serif;"> <p>This is great! <i class='emoji emoji_wink' title=':wink:'></i></p> </div> </body> </html>

and this with 0.0.8:

undefined<undefinedhundefinedtundefinedmundefinedlundefined>undefined undefined<undefinedhundefinedeundefinedaundefineddundefined>undefined undefined<undefinedmundefinedeundefinedtundefinedaundefined undefinedhundefinedtundefinedtundefinedpundefined-undefinedeundefinedqundefineduundefinediundefinedvundefined=undefined"undefinedCundefinedoundefinednundefinedtundefinedeundefinednundefinedtundefined-undefinedTundefinedyundefinedpundefinedeundefined"undefined undefinedcundefinedoundefinednundefinedtundefinedeundefinednundefinedtundefined=undefined"undefinedtundefinedeundefinedxundefinedtundefined/undefinedhundefinedtundefinedmundefinedlundefined;undefined undefinedcundefinedhundefinedaundefinedrundefinedsundefinedeundefinedtundefined=undefineduundefinedtundefinedfundefined-undefined8undefined"undefined>undefined undefined<undefinedsundefinedtundefinedyundefinedlundefinedeundefined undefinedtundefinedyundefinedpundefinedeundefined=undefined"undefinedtundefinedeundefinedxundefinedtundefined/undefinedcundefinedsundefinedsundefined"undefined undefinedsundefinedtundefinedyundefinedlundefinedeundefined=undefined"undefineddundefinediundefinedsundefinedpundefinedlundefinedaundefinedyundefined:undefinednundefinedoundefinednundefinedeundefined;undefined"undefined>undefined undefined<undefined!undefined-undefined-undefined undefinedPundefined undefined{undefinedmundefinedaundefinedrundefinedgundefinediundefinednundefined-undefinedtundefinedoundefinedpundefined:undefined0undefined;undefinedmundefinedaundefinedrundefinedgundefinediundefinednundefined-undefinedbundefinedoundefinedtundefinedtundefinedoundefinedmundefined:undefined0undefined;undefined}undefined undefined-undefined-undefined>undefined undefined<undefined/undefinedsundefinedtundefinedyundefinedlundefinedeundefined>undefined undefined<undefined/undefinedhundefinedeundefinedaundefineddundefined>undefined undefined<undefinedbundefinedoundefineddundefinedyundefined undefineddundefinediundefinedrundefined=undefined"undefinedrundefinedtundefinedlundefined"undefined>undefined undefined<undefineddundefinediundefinedvundefined undefinediundefineddundefined=undefined"undefineddundefinediundefinedvundefinedtundefinedaundefinedgundefineddundefinedeundefinedfundefinedaundefineduundefinedlundefinedtundefinedwundefinedrundefinedaundefinedpundefinedpundefinedeundefinedrundefined"undefined undefinedsundefinedtundefinedyundefinedlundefinedeundefined=undefined"undefinedfundefinedoundefinednundefinedtundefined-undefinedsundefinediundefinedzundefinedeundefined:undefined1undefined2undefinedpundefinedtundefined;undefinedcundefinedoundefinedlundefinedoundefinedrundefined:undefined#undefined0undefined0undefined0undefined0undefined0undefined0undefined;undefinedbundefinedaundefinedcundefinedkundefinedgundefinedrundefinedoundefineduundefinednundefineddundefined-undefinedcundefinedoundefinedlundefinedoundefinedrundefined:undefined#undefinedFundefinedFundefinedFundefinedFundefinedFundefinedFundefined;undefinedfundefinedoundefinednundefinedtundefined-undefinedfundefinedaundefinedmundefinediundefinedlundefinedyundefined:undefinedCundefinedaundefinedlundefinediundefinedbundefinedrundefinediundefined,undefinedAundefinedrundefinediundefinedaundefinedlundefined,undefinedHundefinedeundefinedlundefinedvundefinedeundefinedtundefinediundefinedcundefinedaundefined,undefinedsundefinedaundefinednundefinedsundefined-undefinedsundefinedeundefinedrundefinediundefinedfundefined;undefined"undefined>undefined undefined<undefinedpundefined>undefinedTundefinedhundefinediundefinedsundefined undefinediundefinedsundefined undefinedgundefinedrundefinedeundefinedaundefinedtundefined!undefined undefined:undefinedwundefinediundefinednundefinedkundefined:undefined<undefined/undefinedpundefined>undefined undefined<undefined/undefineddundefinediundefinedvundefined>undefined undefined<undefined/undefinedbundefinedoundefineddundefinedyundefined>undefined undefined<undefined/undefinedhundefinedtundefinedmundefinedlundefined>undefined
dbaq commented 8 years ago

thanks for the feedback, i'll look into it asap.

bogdandrumen commented 8 years ago

I ran into the same problem with 0.0.8. I fixed it by injecting emojiConfigProvider and add at least 1 alias. Hope it helps.

dbaq commented 8 years ago

Very helpful thank you, I pushed and published a fix. It was a regression due to the last PR I accepted, sorry about that. Let me know if it works for you guys.