Menu

Default EventTypes must not rewrite ?

Help
kmcom
2012-08-23
2013-04-29
  • kmcom

    kmcom - 2012-08-23

    Must not rewrite default EventTypes("load", "errror") ?
    I encountered the infinite loop as follows.

    1)
    I modified default EventTypes as follows.

    default:  log4javascript.setEventTypes();
    modified: log4javascript.setEventTypes();

    2)
    Then, I encountered the infinite loop, handleError()->dispatchEvent()->handleError()…

    function handleError(message, exception) {          
        logLog.error(message, exception);       
        log4javascript.dispatchEvent("error", { "message": message, "exception": exception });      
    }
    
    dispatchEvent: function(eventType, eventArgs) {         
        if (array_contains(this.eventTypes, eventType)) {       
        var listeners = this.eventListeners[eventType]; 
        for (var i = 0, len = listeners.length; i < len; i++) { 
            listeners[i](this, eventType, eventArgs);
        }   
        } else {
        handleError("log4javascript.EventSupport [" + this + "]: dispatchEvent: no event called '" + eventType + "'");  
        }       
    }
    
    • log4javascript 1.4.2
     
  • Tim Down

    Tim Down - 2012-09-08

    It's a fair point that it can get into an infinite loop, but since I'd never really intended for the setEventTypes() method to be used except internally and you could just use  log4javascript.setEventTypes(), I'm not too worried.
    How do you want to use it?

     

Log in to post a comment.

Want the latest updates on software, tech news, and AI?
Get latest updates about software, tech news, and AI from SourceForge directly in your inbox once a month.