Sorry! Meniks toi mun yks kommentti väärään kohtaan. Sitä mitä tarkoitin oli se, että koko tää blokki:

$query = "SELECT GROUP_CONCAT(tunnus) tunnukset
+                      FROM varastopaikat
+                      WHERE yhtio = '{$kukarow['yhtio']}'
+                      AND tyyppi != 'P'
+                      AND toimipaikka = '{$kukarow['toimipaikka']}'";
+            $toimipaikka_varasto_res = pupe_query($query);
+            $toimipaikka_varasto_row = mysql_fetch_assoc($toimipaikka_varasto_res);
+
+            if ($toimipaikka_varasto_row['tunnukset'] == '' and !empty($kukarow['toimipaikka'])) {
+              $query = "SELECT GROUP_CONCAT(tunnus) tunnukset
+                        FROM varastopaikat
+                        WHERE yhtio = '{$kukarow['yhtio']}'
+                        AND tyyppi != 'P'
+                        AND toimipaikka = 0";
+              $toimipaikka_varasto_res = pupe_query($query);
+              $toimipaikka_varasto_row = mysql_fetch_assoc($toimipaikka_varasto_res);
+            }

on nyt koodattu kahteen kertaan. Se on mielestäni monella tapaa huono juttu. Ensinnäkin on duplikaattikoodia ja toisekseen tietokantaa kuormitetaan turhaan kyselyllä joka me ollaan jo tehty muutamaa riviä ylempänä. 



On 2.6.2014, at 13.47, Sami Luukkonen <notifications@github.com> wrote:

> In tuote.php:
> 
> > +
> > +          if ($_tp_kasittely) {
> > +            $query = "SELECT GROUP_CONCAT(tunnus) tunnukset
> > +                      FROM varastopaikat
> > +                      WHERE yhtio = '{$kukarow['yhtio']}'
> > +                      AND tyyppi != 'P'
> > +                      AND toimipaikka = '{$kukarow['toimipaikka']}'";
> > +            $toimipaikka_varasto_res = pupe_query($query);
> > +            $toimipaikka_varasto_row = mysql_fetch_assoc($toimipaikka_varasto_res);
> > +
> > +            if ($toimipaikka_varasto_row['tunnukset'] == '') {
> > +              $query = "SELECT GROUP_CONCAT(tunnus) tunnukset
> > +                        FROM varastopaikat
> > +                        WHERE yhtio = '{$kukarow['yhtio']}'
> > +                        AND tyyppi != 'P'
> > +                        AND toimipaikka = 0";
> 
> Vaikka tuossa tehdään kaks samanlaista querya, niin koodi on selkeää ja uusi kooderi varmasti pääsee heti tuohon käsiksi.
> 
> —
> Reply to this email directly or view it on GitHub.
>