UnitApi / js

message.js

// var HomeMessage = function (name, success, error) {
//     this.name = name;
//     this.success = success;
//     this.error = error;
//
//     this.create = function (data) {
//         restSubmit(this.name, 'GET', data, this.success, this.error);
//     }
//
// };


var Message = function (selector) {
    this.selector = selector;


    this.add = function (message) {
        console.log(message);

console.log looks like debug code. Are you sure you do not want to remove it?

        var classname = null;

        if (typeof classname !== 'string') {
            classname = 'home-messages';
        }
        handle = document.getElementsByClassName(classname)

The variable handle seems to be never declared. Assigning variables without defining them first makes them global. If this was intended, consider making it explicit like using window.handle.

        if (handle)

            var node = document.createElement("LI");                 // Create a <li> node

Curly braces around statements make for more readable code and help prevent bugs when you add further statements.

        var textnode = document.createTextNode(message);         // Create a text node
        node.appendChild(textnode);

The variable node does not seem to be initialized in case handle on line 25 is false. Are you sure this can never be the case?

        document.getElementsByClassName(classname)[0].appendChild(node);
    }
}


function AddMessage(text, classname) {
    var message = new Message(classname);
    message.add(text);
}