Issues (171)

message.js (2 issues)

1
// var HomeMessage = function (name, success, error) {
2
//     this.name = name;
3
//     this.success = success;
4
//     this.error = error;
5
//
6
//     this.create = function (data) {
7
//         restSubmit(this.name, 'GET', data, this.success, this.error);
8
//     }
9
//
10
// };
11
12
13
var Message = function (selector) {
14
    this.selector = selector;
15
16
17
    this.add = function (message) {
18
        console.log(message);
19
        var classname = null;
20
21
        if (typeof classname !== 'string') {
22
            classname = 'home-messages';
23
        }
24
        handle = document.getElementsByClassName(classname)
0 ignored issues
show
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.
Loading history...
25
        if (handle)
26
27
            var node = document.createElement("LI");                 // Create a <li> node
28
        var textnode = document.createTextNode(message);         // Create a text node
29
        node.appendChild(textnode);
0 ignored issues
show
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?
Loading history...
30
        document.getElementsByClassName(classname)[0].appendChild(node);
31
    }
32
}
33
34
35
function AddMessage(text, classname) {
36
    var message = new Message(classname);
37
    message.add(text);
38
}
39