literat / srazvs

www/js/function.js

/**

 * Generate a new password, which may then be copied to the form

 * with suggestPasswordCopy().

 *

 * @param   string   the form name

The parameter string does not exist. Did you maybe forget to remove this comment?

 *

 * @return  boolean  always true

 */

function suggestPassword() {

    // restrict the password to just letters and numbers to avoid problems:

    // "editors and viewers regard the password as multiple words and

    // things like double click no longer work"

    var pwchars = "abcdefhjmnpqrstuvwxyz23456789ABCDEFGHJKLMNPQRSTUVWYXZ";

    var passwordlength = 16;    // do we want that to be dynamic?  no, keep it simple :)

    var passwd = document.getElementById('generated_pw');

    passwd.value = '';


    for ( i = 0; i < passwordlength; i++ ) {

The variable i 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.i.

        passwd.value += pwchars.charAt( Math.floor( Math.random() * pwchars.length ) )

    }

    return passwd.value;

}



/**

 * Copy the generated password (or anything in the field) to the form

 *

 * @param   string   the form name

The parameter string does not exist. Did you maybe forget to remove this comment?

 *

 * @return  boolean  always true

 */

function suggestPasswordCopy() {

    document.getElementById('text_pma_pw').value = document.getElementById('generated_pw').value;

    document.getElementById('text_pma_pw2').value = document.getElementById('generated_pw').value;

    return true;

}


/**

 * Opens calendar window.

 *

 * @param   string      calendar.php parameters

The parameter string does not exist. Did you maybe forget to remove this comment?

 * @param   string      form name

The parameter string has already been documented on line 41. The second definition is ignored.

 * @param   string      field name

The parameter string has already been documented on line 41. The second definition is ignored.

 * @param   string      edit type - date/timestamp

The parameter string has already been documented on line 41. The second definition is ignored.

 */

function openCalendar(params, form, field, type) {

    window.open("../calendar/mini_calendar.php?" + params, "calendar", "width=400,height=200,status=yes");

    dateField = eval("document." + form + "." + field);

The variable dateField 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.dateField.

Calls to eval are slow and potentially dangerous, especially on untrusted code. Please consider whether there is another way to achieve your goal.

    dateType = type;

The variable dateType 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.dateType.

}


/**

 * Formats number to two digits.

 *

 * @param   int number to format.

The parameter int does not exist. Did you maybe forget to remove this comment?

 * @param   string type of number

The parameter string does not exist. Did you maybe forget to remove this comment?

 */

function formatNum2(i, valtype) {

    f = (i < 10 ? '0' : '') + i;

The variable f 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.f.

    if (valtype && valtype != '') {

        switch(valtype) {

            case 'month':

                f = (f > 12 ? 12 : f);

                break;


            case 'day':

                f = (f > 31 ? 31 : f);

                break;


            case 'hour':

                f = (f > 24 ? 24 : f);

                break;


            default:

The default case is not the last statement in this switch statement. For the sake of readability, you might want to move it to the end of the statement.

            case 'second':

            case 'minute':

                f = (f > 59 ? 59 : f);

                break;

        }

    }


    return f;

}


/**

 * Formats number to two digits.

 *

 * @param   int number to format.

The parameter int does not exist. Did you maybe forget to remove this comment?

 * @param   int default value

The parameter int has already been documented on line 88. The second definition is ignored.

 * @param   string type of number

The parameter string does not exist. Did you maybe forget to remove this comment?

 */

function formatNum2d(i, default_v, valtype) {

    i = parseInt(i, 10);

    if (isNaN(i)) return default_v;

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

    return formatNum2(i, valtype)

}


/**

 * Formats number to four digits.

 *

 * @param   int number to format.

The parameter int does not exist. Did you maybe forget to remove this comment?

 */

function formatNum4(i) {

    i = parseInt(i, 10)

    return (i < 1000 ? i < 100 ? i < 10 ? '000' : '00' : '0' : '') + i;

}


/**

 * Initializes calendar window.

 */

function initCalendar() {

    if (!year && !month && !day) {

If you intend to check if the variable month is declared in the current environment, consider using typeof month === "undefined" instead. This is safe if the variable is not actually declared.

If you intend to check if the variable year is declared in the current environment, consider using typeof year === "undefined" instead. This is safe if the variable is not actually declared.

If you intend to check if the variable day is declared in the current environment, consider using typeof day === "undefined" instead. This is safe if the variable is not actually declared.

        /* Called for first time */

        if (window.opener.dateField.value) {

            value = window.opener.dateField.value;

The variable value 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.value.

            if (window.opener.dateType == 'datetime' || window.opener.dateType == 'date') {

                if (window.opener.dateType == 'datetime') {

                    parts   = value.split(' ');

The variable parts 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.parts.

                    value   = parts[0];


                    if (parts[1]) {

                        time    = parts[1].split(':');

The variable time 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.time.

                        hour    = parseInt(time[0],10);

The variable hour 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.hour.

                        minute  = parseInt(time[1],10);

The variable minute 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.minute.

                        second  = parseInt(time[2],10);

The variable second 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.second.

                    }

                }

                date        = value.split("-");

The variable date 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.date.

                day         = parseInt(date[2],10);

The variable day 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.day.

                month       = parseInt(date[1],10) - 1;

The variable month 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.month.

                year        = parseInt(date[0],10);

The variable year 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.year.

            } else {

                year        = parseInt(value.substr(0,4),10);

                month       = parseInt(value.substr(4,2),10) - 1;

                day         = parseInt(value.substr(6,2),10);

                hour        = parseInt(value.substr(8,2),10);

                minute      = parseInt(value.substr(10,2),10);

                second      = parseInt(value.substr(12,2),10);

            }

        }

        if (isNaN(year) || isNaN(month) || isNaN(day) || day == 0) {

The variable year does not seem to be initialized in case window.opener.dateField.value on line 114 is false. Are you sure the function isNaN handles undefined variables?

The variable day does not seem to be initialized in case window.opener.dateField.value on line 114 is false. Are you sure the function isNaN handles undefined variables?

The variable month does not seem to be initialized in case window.opener.dateField.value on line 114 is false. Are you sure the function isNaN handles undefined variables?

            dt      = new Date();

The variable dt 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.dt.

            year    = dt.getFullYear();

            month   = dt.getMonth();

            day     = dt.getDate();

        }

        if (isNaN(hour) || isNaN(minute) || isNaN(second)) {

The variable second seems to not be initialized for all possible execution paths. Are you sure isNaN handles undefined variables?

The variable minute seems to not be initialized for all possible execution paths. Are you sure isNaN handles undefined variables?

The variable hour seems to not be initialized for all possible execution paths. Are you sure isNaN handles undefined variables?

            dt      = new Date();

            hour    = dt.getHours();

            minute  = dt.getMinutes();

            second  = dt.getSeconds();

        }

    } else {

        /* Moving in calendar */

        if (month > 11) {

            month = 0;

            year++;

        }

        if (month < 0) {

            month = 11;

            year--;

        }

    }


    if (document.getElementById) {

        cnt = document.getElementById("calendar_data");

The variable cnt 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.cnt.

    } else if (document.all) {

        cnt = document.all["calendar_data"];

    }


    cnt.innerHTML = "";

The variable cnt does not seem to be initialized in case document.all on line 167 is false. Are you sure this can never be the case?

    str = ""

The variable str 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.str.

    //heading table

    str += '<table class="calendar"><tr><th width="50%">';

    str += '<form method="NONE" onsubmit="return 0">';

    str += '<a href="javascript:month--; initCalendar();">&laquo;</a> ';

    str += '<select id="select_month" name="monthsel" onchange="month = parseInt(document.getElementById(\'select_month\').value); initCalendar();">';

    for (i =0; i < 12; i++) {

The variable i 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.i.

        if (i == month) selected = ' selected="selected"';

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

The variable selected 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.selected.

        else selected = '';

        str += '<option value="' + i + '" ' + selected + '>' + month_names[i] + '</option>';

The variable month_names seems to be never declared. If this is a global, consider adding a /** global: month_names */ comment.

    }

    str += '</select>';

    str += ' <a href="javascript:month++; initCalendar();">&raquo;</a>';

    str += '</form>';

    str += '</th><th width="50%">';

    str += '<form method="NONE" onsubmit="return 0">';

    str += '<a href="javascript:year--; initCalendar();">&laquo;</a> ';

    str += '<select id="select_year" name="yearsel" onchange="year = parseInt(document.getElementById(\'select_year\').value); initCalendar();">';

    for (i = year - 25; i < year + 25; i++) {

        if (i == year) selected = ' selected="selected"';

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

        else selected = '';

        str += '<option value="' + i + '" ' + selected + '>' + i + '</option>';

    }

    str += '</select>';

    str += ' <a href="javascript:year++; initCalendar();">&raquo;</a>';

    str += '</form>';

    str += '</th></tr></table>';


    str += '<table class="calendar"><tr>';

    for (i = 0; i < 7; i++) {

        str += "<th>" + day_names[i] + "</th>";

The variable day_names seems to be never declared. If this is a global, consider adding a /** global: day_names */ comment.

    }

    str += "</tr>";


    var firstDay = new Date(year, month, 1).getDay();

    var lastDay = new Date(year, month + 1, 0).getDate();


    str += "<tr>";


    dayInWeek = 0;

The variable dayInWeek 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.dayInWeek.

    for (i = 0; i < firstDay; i++) {

        str += "<td>&nbsp;</td>";

        dayInWeek++;

    }

    for (i = 1; i <= lastDay; i++) {

        if (dayInWeek == 7) {

            str += "</tr><tr>";

            dayInWeek = 0;

        }


        dispmonth = 1 + month;

The variable dispmonth 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.dispmonth.

        if (window.opener.dateType == 'datetime' || window.opener.dateType == 'date') {

            actVal = "" + formatNum4(year) + "-" + formatNum2(dispmonth, 'month') + "-" + formatNum2(i, 'day');

The variable actVal 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.actVal.

        } else {

            actVal = "" + formatNum4(year) + formatNum2(dispmonth, 'month') + formatNum2(i, 'day');

        }

        if (i == day) {

            style = ' class="selected"';

The variable style 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.style.

            current_date = actVal;

The variable current_date 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.current_date.

        } else {

            style = '';

        }

        str += "<td" + style + "><a href=\"javascript:returnDate('" + actVal + "');\">" + i + "</a></td>"

        dayInWeek++;

    }

    for (i = dayInWeek; i < 7; i++) {

        str += "<td>&nbsp;</td>";

    }


    str += "</tr></table>";


    cnt.innerHTML = str;


    // Should we handle time also?

    if (window.opener.dateType != 'date' && !clock_set) {

If you intend to check if the variable clock_set is declared in the current environment, consider using typeof clock_set === "undefined" instead. This is safe if the variable is not actually declared.

        if (document.getElementById) {

            cnt = document.getElementById("clock_data");

        } else if (document.all) {

            cnt = document.all["clock_data"];

        }


        str = '';

        init_hour = hour;

The variable init_hour 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.init_hour.

        init_minute = minute;

The variable init_minute 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.init_minute.

        init_second = second;

The variable init_second 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.init_second.

        str += '<fieldset>';

        str += '<form method="NONE" class="clock" onsubmit="returnDate(\'' + current_date + '\')">';

The variable current_date seems to not be initialized for all possible execution paths.

        str += '<input id="hour"    type="text" size="2" maxlength="2" onblur="this.value=formatNum2d(this.value, init_hour, \'hour\'); init_hour = this.value;" value="' + formatNum2(hour, 'hour') + '" />:';

        str += '<input id="minute"  type="text" size="2" maxlength="2" onblur="this.value=formatNum2d(this.value, init_minute, \'minute\'); init_minute = this.value;" value="' + formatNum2(minute, 'minute') + '" />:';

        str += '<input id="second"  type="text" size="2" maxlength="2" onblur="this.value=formatNum2d(this.value, init_second, \'second\'); init_second = this.value;" value="' + formatNum2(second, 'second') + '" />';

        str += '&nbsp;&nbsp;';

        str += '<input type="submit" value="' + submit_text + '"/>';

The variable submit_text seems to be never declared. If this is a global, consider adding a /** global: submit_text */ comment.

        str += '</form>';

        str += '</fieldset>';


        cnt.innerHTML = str;

        clock_set = 1;

The variable clock_set 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.clock_set.

    }


}


/**

 * Returns date from calendar.

 *

 * @param   string     date text

The parameter string does not exist. Did you maybe forget to remove this comment?

 */

function returnDate(d) {

    txt = d;

The variable txt 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.txt.

    if (window.opener.dateType != 'date') {

        // need to get time

        h = parseInt(document.getElementById('hour').value,10);

The variable h 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.h.

        m = parseInt(document.getElementById('minute').value,10);

The variable m 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.m.

        s = parseInt(document.getElementById('second').value,10);

The variable s 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.s.

        if (window.opener.dateType == 'datetime') {

            txt += ' ' + formatNum2(h, 'hour') + ':' + formatNum2(m, 'minute') + ':' + formatNum2(s, 'second');

        } else {

            // timestamp

            txt += formatNum2(h, 'hour') + formatNum2(m, 'minute') + formatNum2(s, 'second');

        }

    }


    window.opener.dateField.value = txt;

    window.close();

}




//-------------------------------------------------

function getE(id)

{

   if (typeof id != "string")

   {

         return id;

   }

   else if (Boolean(document.getElementById))

   {

         return document.getElementById(id);

   }

   else if (Boolean(document.all))

   {

         return eval("document.all."+id);

Calls to eval are slow and potentially dangerous, especially on untrusted code. Please consider whether there is another way to achieve your goal.

   }

   else if (Boolean(document.ids))

   {

         return eval("document.ids."+id);

Calls to eval are slow and potentially dangerous, especially on untrusted code. Please consider whether there is another way to achieve your goal.

   }

   else

   {

         return null;

   }

}

//-------------------------------------------------

function okno(action,winwidth,winheight,name,scroll,toolbar,status,resizable,menubar) {

	var PROFILE = null;

The assignment to PROFILE seems to be never used. If you intend to free memory here, this is not necessary since the variable leaves the scope anyway.

       

	PROFILE =  window.open ("", name, "toolbar="+toolbar+",width="+winwidth+",height="+winheight+",directories=no,location=no,status="+status+",scrollbars="+scroll+",resizable="+resizable+",menubar="+menubar+"");

        if (PROFILE != null) {

               if (PROFILE.opener == null) {

                   PROFILE.opener = self;

The variable self seems to be never declared. If this is a global, consider adding a /** global: self */ comment.

        	   }

	       PROFILE.location.href = action;

        } 

       

       

}

//-------------------------------------------------

function reokno(action,winwidth,winheight,name,scroll,toolbar,status,resize,menubar) 

This code seems to be duplicated in your project.

	{

	

	window.close();

	

	b = window.opener;

The variable b 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.b.

	a = b.closed;

The variable a 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.a.

				

	if (!a)

	{

		var PROFILE = null;

The assignment to PROFILE seems to be never used. If you intend to free memory here, this is not necessary since the variable leaves the scope anyway.

	       	PROFILE =  self.open ("", "nalezen", "toolbar="+toolbar+",width="+winwidth+",height="+winheight+",directories=no, status="+status+",scrollbars="+scroll+",resize="+resize+",menubar="+menubar+"");

The variable self seems to be never declared. If this is a global, consider adding a /** global: self */ comment.

	        if (PROFILE != null) {

	               if (PROFILE.opener == null) {

	                   PROFILE.opener = self;

	        	   }

		       PROFILE.location.href = action;

	}

        }

}

//-------------------------------------------------

function reoknoclen(action,winwidth,winheight,name,scroll,toolbar,status,resize,menubar) 

This code seems to be duplicated in your project.

	{

	

	window.close();

	

	b = window.opener;

The variable b 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.b.

	a = b.closed;

The variable a 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.a.

				

	if (!a)

	{

		var PROFILE = null;

The assignment to PROFILE seems to be never used. If you intend to free memory here, this is not necessary since the variable leaves the scope anyway.

	       	PROFILE =  self.open ("", "clen", "toolbar="+toolbar+",width="+winwidth+",height="+winheight+",directories=no, status="+status+",scrollbars="+scroll+",resize="+resize+",menubar="+menubar+"");

The variable self seems to be never declared. If this is a global, consider adding a /** global: self */ comment.

	        if (PROFILE != null) {

	               if (PROFILE.opener == null) {

	                   PROFILE.opener = self;

	        	   }

		       PROFILE.location.href = action;

	}

        }

        }

//-------------------------------------------------

function GetElementById(id){

	if (document.getElementById) {

		return (document.getElementById(id));

	} else if (document.all) {

		return (document.all[id]);

	} else {

		if ((navigator.appname.indexOf("Netscape") != -1) && parseInt(navigator.appversion == 4)) {

The variable navigator seems to be never declared. If this is a global, consider adding a /** global: navigator */ comment.

There is no return statement if navigator.appname.indexOf("Netscape") != -1 && parseInt(navigator.appversion == 4) is false. Are you sure this is correct? If so, consider adding return; explicitly.

			return (document.layers[id]);

		}

	}

}

//-------------------------------------------------

function CheckAll(formname, switchid) {

	var ele = document.forms[formname].elements;

	var switch_cbox = GetElementById(switchid);

	for (var i = 0; i < ele.length; i++) {

		var e = ele[i];

		if ( (e.name != switch_cbox.name) && (e.type == 'checkbox') ) {

			e.checked = switch_cbox.checked;

		}

	}

}

//-------------------------------------------------

function stylTextu(formname, policko, styl) {

if(styl == "a")

{

	var adresa = prompt('Zadej adresu odkazu', '');

	var text = prompt('Zadej název odkazu', '');

	document.forms[formname].elements[policko].value += "<"+ styl +" href=\""+ adresa +"\">"+ text +"</"+ styl +">";

}

else

{

	var text = prompt('Zadej text', '');

The variable text already seems to be declared on line 411. Consider using another variable name or omitting the var keyword.

	document.forms[formname].elements[policko].value += "<"+ styl +">"+ text +"</"+ styl +">";

}

}

//-------------------------------------------------

function anketa_check_form(formname,moznost) {

	var ele = formname.moznost;

var chyba = "Nevybrali jste žádnou možnost!";

		for (var i = 0; i < ele.length; i++) {

			var e = ele[i];

			if (e.checked == true){

			var chyba = "";

The variable chyba already seems to be declared on line 423. Consider using another variable name or omitting the var keyword.

			}

		}

		if (chyba != ""){

			alert (chyba);

			return false;

		}

		else return true;

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

}

//-------------------------------------------------

function confirmation(adresa, hlaska) 

          {

          var potvrdit = confirm (hlaska);

          if (potvrdit)

               {

               //self.location.href = adresa;

			   window.location.replace(adresa);

			   }

	     }

//-------------------------------------------------

function validate_email(field, alerttxt)

		{

			with (field){

The use of the with statement is discouraged as it makes your code harder to understand and may not be forward compatible with newer versions of javascript.

				apos=value.indexOf("@");

				dotpos=value.lastIndexOf(".");

				if (apos<1||dotpos-apos<2){

					alert(alerttxt);

					return false;

				}

				else {return true;}

			}

		}

//-------------------------------------------------

function validate_empty(field, alerttxt)

		{

			with (field){

The use of the with statement is discouraged as it makes your code harder to understand and may not be forward compatible with newer versions of javascript.

				if (value == ""){

					alert(alerttxt);

					return false;

				}

				else {return true;}

			}

		}

//-------------------------------------------------

function UkazHide(div,stav)

{

   var d = getE(div);

   var _stav = stav;


   if (d == null) return;

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

   if (stav == 3)

   {

         if (d.style.display == "") _stav = 0;

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

         else                       _stav = 1;

   }


   if (_stav == 0)

   {

         d.style.visibility = "hidden";

         d.style.display    = "none";

   }

   else

   {

         d.style.visibility = "visible";

         d.style.display    = "";

   }

}

//-------------------------------------------------

function ZmenSelect(ele,value)

{

   var d = getE(ele);

   var v = value;

   d.selectedIndex = v;

}

/*

function checkAllCheckboxes(field)

{

	for (i = 0; i < field.length; i++)

		field[i].checked = true ;

}


function uncheckAllCheckboxes(field)

{

	for (i = 0; i < field.length; i++)

		field[i].checked = false ;

}


function checkAll(theForm)

{

	for (i=0;i<theForm.elements.length;i++)

		if (theForm.elements[i].name.indexOf('checker') !=-1)

		theForm.elements[i].checked = true;

}


function uncheckAll(theForm)

{

	for (i=0;i<theForm.elements.length;i++)

		if (theForm.elements[i].name.indexOf('checker') !=-1)

		theForm.elements[i].checked = false;

}*/


function select(boolean) {

	var theForm = document.checkerForm;

	for (i=0; i<theForm.elements.length; i++) {

The variable i 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.i.

		if (theForm.elements[i].name=='checker[]')

		theForm.elements[i].checked = boolean;

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

	}

}