Awesome!

Great catch, I will work on merging it in soon!

Thanks for contributing to ngUpload!

Tom


On Wed, Jul 24, 2013 at 7:28 PM, trov-codystebbins <notifications@github.com
> wrote:

> In ng-upload.js:
>
> >                          try {
> > -                            content = $.parseJSON(iframe.contents().find('body').text());
> > +                            content = $.parseJSON(content);
>
> Before you used .text() here instead of .html(). This seems unnecessary
> since if the response is not JSON $.parseJSON would fail on .html() or
> .text(). Instead I pass the innerHTML, which is equivalent to $.html(),
> since innerText is not cross browser like innerHTML.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/twilson63/ngUpload/pull/56/files#r5387371>
> .
>



-- 
Tom Wilson
Jack Russell Software Company Division of CareKinesis
494 Wando Park Blvd
Mount Pleasant, SC  29464
Phone: 843-881-2171
Email: tom@jackhq.com
Web: http://www.jackhq.com
Calendar:
http://www.google.com/calendar/embed?src=tom%40jackrussellsoftware.com&ctz=America/New_York<http://www.jackhq.com/calendar>

This e-mail may contain information that is confidential, privileged or
otherwise protected from disclosure by the Health Insurance Portability and
Accountability Act (HIPAA) and other state and federal laws.  This
information is intended only for the individual names above.  Any review,
use disclosure or dissemination of this material is strictly prohibited.
If you receive this information in error, please notify CareKinesis
immediately at 888-974-2763 and delete the original at once.