[
https://issues.apache.org/jira/browse/SHINDIG-1725?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13225325#comment-13225325
]
[email protected] commented on SHINDIG-1725:
--------------------------------------------------------
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4247/
-----------------------------------------------------------
Review request for shindig.
Summary
-------
Remove osapi.container.util.warn because it only serves to cause issues in IE.
Replace it with gadgets.warn instead.
This addresses bug SHINDIG-1725.
https://issues.apache.org/jira/browse/SHINDIG-1725
Diffs
-----
http://svn.apache.org/repos/asf/shindig/trunk/content/samplecontainer/examples/commoncontainer/index.html
1298060
http://svn.apache.org/repos/asf/shindig/trunk/content/samplecontainer/examples/commoncontainer/viewController.js
1298060
http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/container.site.gadget/gadget_site.js
1298060
http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/container.util/util.js
1298060
http://svn.apache.org/repos/asf/shindig/trunk/features/src/test/javascript/features/container/gadget_site_test.js
1298060
Diff: https://reviews.apache.org/r/4247/diff
Testing
-------
Modified the sample common container to allow me to hit the code paths I'm
testing. These changes are part of the patch and I can drop them if we really
don't want them going back to Shindig.
Thanks,
Stanton
> osapi.container.util.warn causes errors when used in Internet Explorer
> ----------------------------------------------------------------------
>
> Key: SHINDIG-1725
> URL: https://issues.apache.org/jira/browse/SHINDIG-1725
> Project: Shindig
> Issue Type: Bug
> Components: Javascript
> Affects Versions: 2.5.0
> Environment: Internet Explorer 8
> Reporter: Stanton Sievers
> Assignee: Stanton Sievers
> Fix For: 2.5.0
>
> Original Estimate: 2h
> Remaining Estimate: 2h
>
> osapi.container.util.warn uses "console" to do logging. IE complains when
> "console" is undefined even though the code is checking truthiness.
> osapi.container.util.warn is only used by
> osapi.container.GadgetSite.prototype.navigateTo and the proposed fix would be
> to remove osapi.container.util.warn completely and replace it with
> gadgets.warn, which handles the console object appropriately across browsers.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
