#71 Remove commented out code and unused code

web/tupa/TehtavanMaaritys.py

@@ -72,7 +72,6 @@ def is_kaava(s):
         numero = 1
         for h in haku:
             muuttujat[h.group(1)] = Decimal(numero)
-            # numero+=1
         tulos = laskeTaulukko([[kaava]], muuttujat)
         if tulos[0][0] == None or tulos[0][0] == "S":
             return False
@@ -903,7 +902,6 @@ def puhdasKaavaForm(posti, data, prefix):
         lataa_parametrit(state, data, prefix, "_pk")
     else:
         state = posti.copy()
-    # formi.update( field(state,"parhaan_haku",prefix+"_pk")  )
 
     errors = ""
     if state and prefix in state.keys() and state[prefix] == "pk":
@@ -998,15 +996,6 @@ def tehtavanMaaritysForm(
     formidata = []
     # luodaan uusi tehtava jos vanhaa ei loydy
     if len(data.items()) == 0:
-        """
-        if not posti:
-                data['#1']={'sarja' : sarja_id,
-                                        'kaava': 'ss' ,
-                                        'nimi': '' ,
-                                        'jarjestysnro':suurin_jarjestysnro+1,
-                                        'osa_tehtavat' : { '#1': { "nimi": "a","tyyppi": "" }  } }
-        else :
-        """
         data["#1"] = {
             "sarja": sarja_id,
             "kaava": "ss",
@@ -1158,8 +1147,6 @@ def tehtavanMaaritysForm(
                     )
                 data[k][fk] = value
             formidata.append(("osa_tehtavat", ot_formit))
-            # formidata.append( ('tehtava_id',k) )
-            # formidata.append( ('kisa_nimi',) )
             tags.update(formidata)
     return render_to_string("tupa/forms/tehtava.html", tags)
 

web/tupa/TulosLaskin.py

@@ -166,10 +166,7 @@ def luoOsatehtavanKaava(ot_lause, parametrit):
         ot_lause = re.sub("muk" + r"(?!\w+)", "..mukana", ot_lause)
         # Muunnos "suor" -> kaikkien vartioiden lasketut suoritukset
         try:
-            vartion_kaava = parametrit[
-                "vartion_kaava"
-            ]  # .filter(nimi="vartion_kaava")[0].arvo
-            # vartion_kaava=re.sub("vartio"+r"(?!\w+)", str(v.nro) ,vartion_kaava)
+            vartion_kaava = parametrit["vartion_kaava"]
             for p_nimi, p_arvo in parametrit.items():
                 vartion_kaava = re.sub(p_nimi + r"(?!\w+)", p_arvo, vartion_kaava)
             ot_lause = re.sub(
@@ -270,7 +267,6 @@ def laskeSarja(sarja, syotteet, vartiot=None, tehtavat=None):
     Taulukon ensimmäisissä sarakkeissa on vartio tai tehtävä objekteja muissa pisteitä.
     Taulukon vasemmassa ylänurkassa on sarjan objekti
     """
-    # syotteetr=Syote.objects.all() #(maarite__osa_tehtava__tehtava__sarja=sarja )
 
     if not vartiot:
         vartiot = sarja.vartio_set.all()
@@ -294,7 +290,6 @@ def laskeSarja(sarja, syotteet, vartiot=None, tehtavat=None):
             tehtSyotteet = syotteet.filter(
                 maarite__osa_tehtava__tehtava=tehtavat[t]
             ).filter(vartio=vartiot[i])
-            # syotteet= vartiot[i].syote_set.filter(maarite__osa_tehtava__tehtava=tehtavat[t])
             for s in tehtSyotteet:
                 if s.arvo == "e":
                     tekematta = True

web/tupa/laskentatyypit.py

@@ -12,10 +12,6 @@ def decimal_repr(self):
     return str(self.quantize(Decimal("0.1"), rounding=ROUND_HALF_UP))
 
 
-# Decimal.__repr__ = decimal_repr
-# Decimal.__str__ = decimal_uni
-
-
 class SequenceOperations:
     def __add__(self, other):
         return self.operate_to_all(lambda a, b: a + b, other)
@@ -289,8 +285,6 @@ def karsi(lista, lfunktio):
                     varvi.append(l[index])
             else:
                 varvi.append(l)
-            # if type(varvi[-1])==unicode :
-            #        varvi.pop(-1)
         if tavaraa == 0 and index > 0:
             break
         index += 1
@@ -321,8 +315,6 @@ def listaksi(a, *opt):
         joukkio = [DictDecimal(joukkio)]
     elif type(joukkio) == str:
         return joukkio
-    # elif type( joukkio )== MathDict:
-    # return joukkio
     if type(joukkio) == list:
         lista = []
         for v in joukkio:
@@ -369,7 +361,6 @@ def run_dict(list, funktio, *param):
 
         if list:
             rValue[k] = karsi(listaksi(*parametrit), funktio)
-            # rValue[k] = funktio(listaksi(*parametrit))
         else:
             try:
                 rValue[k] = funktio(*parametrit)

web/tupa/taulukkolaskin.py

@@ -67,7 +67,6 @@ def laske(lauseke, m={}, funktiot={}):
         r"num('\g<1>')",
         lause,
     )
-    # lause=re.sub(r"((?<![^-+*/(,[])-?\d+([.]\d+)?)(?![\.0-9])",r"num('\g<1>')",lause)
 
     # Korvataan muuttujien nimet oikeilla muuttujilla:
     lause = re.sub(r"\.([a-zA-Z_]\w*)(?=\.)", r"['\g<1>']", lause)  # .x. -> [x].

web/tupa/tests.py

@@ -116,9 +116,6 @@ def testMinimiFunktio(self):
     def testMedFunktio(self):
         assert laske("med(a)", {"a": Decimal("4")}) == Decimal("4")
 
-    # def testMedFunktio2(self):
-    #    assert laske('med(a)', {'a': {'a' : Decimal("4"),'b' : Decimal("5"),'c' : Decimal("6")} }) == Decimal("5")
-    # Poistettu, koska nykyinen toteutsu ei toimi n�in, muutetaan vastaavaksi joskus...
     def testListaFunktio(self):
         assert laske("min([a,3,1])", {"a": Decimal("100")}) == Decimal("1")
 
@@ -299,8 +296,6 @@ def testTehtavanUudelleenTallennus(self):
             settings.DEBUG = False
             self.TAUSTALASKENTA = settings.TAUSTALASKENTA
             settings.TAUSTALASKENTA = None
-            posti_vapaa = {}
-            posti_puhdas = {}
             for s in Sarja.objects.select_related().all():
                 for t in s.tehtava_set.all():
                     # Parsitaan post data html sivusta:

web/tupa/views.py

@@ -500,8 +500,6 @@ def syotaTehtava(request, kisa_nimi, tehtava_id, talletettu=None, tarkistus=None
     validi = True
 
     tehtavan_syotteet = Syote.objects.filter(maarite__osa_tehtava__tehtava=tehtava)
-    if tehtavan_syotteet:
-        hot = 69  # Viettelee syotteet kannasta.
 
     tehtava.svirhe = 0
     for v in vartiot:
@@ -1266,8 +1264,6 @@ def tehtavanVaiheet(request, kisa_nimi, tehtava_id, vartio_id=None):
     enableLogging()
     clearLoki()
     tehtavan_syotteet = Syote.objects.filter(maarite__osa_tehtava__tehtava=tehtava)
-    if tehtavan_syotteet:
-        hot = 69  # Viettelee syotteet kannasta.
 
     laskeSarja(
         tehtava.sarja, tehtavan_syotteet, Vartio.objects.filter(id=vartio_id), [tehtava]