LSS-Manager/lss-manager-v2

Code Qualität

Closed this issue · 2 comments

BITTE geht euren Code mit JSLint durch.
Von mir aus kann ich auch nachher einen Pull Request mit den Verbesserungen machen, aber im Moment habt ihr zahlreiche Fehler im Code.
Ja, Fehler, nicht nur designtechnisch.

var org_building_maps_draw = building_maps_draw;
building_maps_draw = function (e) {
    //org_building_maps_draw(e);
    var t = L.marker([e.latitude, e.longitude], {
        title: e.name,
        icon: icon_empty,
        riseOnHover: true
    }).bindLabel(e.name).addTo(map);
    t.building_id = e.id, "undefined" != typeof e.opacity && t.setOpacity(e.opacity), iconMapGenerate(e.building_marker_image, t), t.on("click", function () {
        lightboxOpen("/buildings/" + e.id)
    }), building_markers.push(t)
    $.each(building_markers, function (key, value) {
        value.hideLabel();
        value.unbindLabel();
        cars = car_list(value.building_id);
        data = value.options.title;
        if (cars.length > 0) {
            data += '&nbsp;<i class="fa fa-car"></i>' + cars.length;
            data += car_list_printable(cars);
        }
        value.bindLabel(data, {zIndex: 999});
    });
}

Hier geb ich auf. Keine Ahnung, was das soll, aber das kann so nicht stimmen.