[jira] [Commented] (MYFACES-3902) jsf.js: error handling output improvement

["Werner Punz (JIRA)" <de...@myfaces.apache.org>]

    [ https://issues.apache.org/jira/browse/MYFACES-3902?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14040516#comment-14040516 ] 

Werner Punz commented on MYFACES-3902:
--------------------------------------

Thanks for reporting the issue, I will work on it sometime this week, as my time permits.


> jsf.js: error handling output improvement
> -----------------------------------------
>
>                 Key: MYFACES-3902
>                 URL: https://issues.apache.org/jira/browse/MYFACES-3902
>             Project: MyFaces Core
>          Issue Type: Bug
>          Components: General
>    Affects Versions: 2.2.3
>         Environment: Snapshot code 2.2.3 from trunk + Icefaces 4/Primefaces 4 
> Firefox >=18
>            Reporter: Krashan Brahmanjara
>
> Myfaces core do not show real javascipt errors.
> In _Dom.js there is a catch block in rows 192-203.
> Unofortunately I see two errors
> - Icefaces js has exceptions without message and description a message is directly a 'e' object. So an error 'invalid property' is printed to console
> - development mode is ignored by this catch block, in reverse order checking too because jsf.ajax.getProjectStage() is not initialized
> Workaround - because development mode is recognized we can only show an alert
> {noformat}
>         } catch (e) {
> /*
>             if(window.console && window.console.error) {
>                //not sure if we
>                //should use our standard
>                //error mechanisms here
>                //because in the head appendix
>                //method only a console
>                //error would be raised as well
>                 if (e.message) {
>                   window.console.error(e.message);
>                 } else {
>                     if (e.description) {
>                       window.console.error(e.description);
>                     } else {
>                       window.console.error(e);
>                     }
>                }
>             }
>             if(jsf.ajax.getProjectStage() === "Development") {
> */
> 				if (e.message) {
> 				  alert("Error in evaluated javascript:"+ (e.message));
> 				} else {
> 					if (e.description) {
> 					  alert("Error in evaluated javascript:"+ (e.description));
> 					} else {
> 					  alert("Error in evaluated javascript:"+ (e));
> 					}
> 				}
> /*
>             }
> */
>        } finally {
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.2#6252)