[BETEPAH]

а почему

this.className = (this.className == 'on' ? 'off' : 'square')

а не

this.className = (this.className == 'on' ? 'off' : 'on')

или

this.className = (this.className == 'accordion-toggle on' ? 'accordion-toggle off' : 'accordion-toggle on')

?