Mikä tässä koodissa vikana?

apuatarttis

Joo elikkä kyseessä on siis php systeemi jolla toiset ovat lähetelleet minulle sähköpostiin tietoja. Toimi hyvin melko pitkään, ja sitten ihan yhtäkkiä ei toimikkaan? Sähköpostiin ei tule enää tietoja, eikä edes viestiä? Tohon viestiin tosiaan on toi s-postiosoite muokattu, eli älkää vastatko tähän että olen syöttänyt väärän sähköpostiosoitteen :-D

Jos joku ekspertti löytäisi tuosta vian? Voin kyllä lähetellä myös tota "lomakepuoltakin" tänne jos tarvitsee. Eli siis sitä tiedostoa, jonka kentät nämä kaverit täyttävät ja siten lähettävät.

7

517

    Vastaukset

    Anonyymi (Kirjaudu / Rekisteröidy)
    5000
    • keksa1

      ja ihan alkuun pistät testailun ajaksi

      error_reporting(E_ALL);
      ini_set('display_errors', '1');

      ja sitten ajat skriptiä ihan tuohon

      $viesti = mail("[email protected]", "Otsikko", $tieto1);

      asti vaan. Pitäis tulla kaikki virheet selaimelle asti.

      En usko että koodi on mennyt mystisesti rikki, vaan että postien lähetyksessä kusee joku esim. palomuuri

      • apuatarttis

        Joo, elikkä koitin laitella koodia sellaiseksi mitä tuossa sanoit. Näyttää tältä:


        Nyt kun tekee lähetyksen niin tulee seuraavanlainen ilmoitus:
        Notice: Undefined variable: tieto1 in /home/a7460772/public_html/korjailulahetys.php on line 6

        Kokeilin muuttaa line 6:sta seuraavasti:
        $tieto1 .= $tunnus . ": " . $arvo . "\n";
        --> $tieto1 = $tunnus . ": " . $arvo . "\n";

        Elikkäs pisteen poisto ennen yhtäsuuruusmerkkiä. Valitettavasti olin jo kokeillut tuota pisteen poistamista aikaisemminkin, kun itse yritin korjailla, eli hirveän luottavainen en ollut. No, nyt kun lähetystä pistää tapahtumaan, niin lähetyssivu vaan näkyy valkoisena ilman mitään erroreita tms. mutta mitään ei löydy sähköpostista.

        Uusin mahdollisesti koko koodin erilaiseksi, kun tämä vaikuttaa olevan aika surkea tekele. Siinä kuitenkin menee aikansa, joten mitä mun pitäis nyt ton koodin kanssa tehdä vielä? Vai onko ongelma niin mystinen, että ei vastausta löydy :-D


      • kanssakoodari
        apuatarttis kirjoitti:

        Joo, elikkä koitin laitella koodia sellaiseksi mitä tuossa sanoit. Näyttää tältä:


        Nyt kun tekee lähetyksen niin tulee seuraavanlainen ilmoitus:
        Notice: Undefined variable: tieto1 in /home/a7460772/public_html/korjailulahetys.php on line 6

        Kokeilin muuttaa line 6:sta seuraavasti:
        $tieto1 .= $tunnus . ": " . $arvo . "\n";
        --> $tieto1 = $tunnus . ": " . $arvo . "\n";

        Elikkäs pisteen poisto ennen yhtäsuuruusmerkkiä. Valitettavasti olin jo kokeillut tuota pisteen poistamista aikaisemminkin, kun itse yritin korjailla, eli hirveän luottavainen en ollut. No, nyt kun lähetystä pistää tapahtumaan, niin lähetyssivu vaan näkyy valkoisena ilman mitään erroreita tms. mutta mitään ei löydy sähköpostista.

        Uusin mahdollisesti koko koodin erilaiseksi, kun tämä vaikuttaa olevan aika surkea tekele. Siinä kuitenkin menee aikansa, joten mitä mun pitäis nyt ton koodin kanssa tehdä vielä? Vai onko ongelma niin mystinen, että ei vastausta löydy :-D

        Se piste tarkoittaa että merkkijonomuuttujaan sijoitettava arvo liitetään merkkijonon loppuun. Nyt kun poistit sen niin arvoista vain viimeinen jää muuttujaan.

        Virheilmoitus kertoi että tieto1-muuttujaa ei ole määritetty. Ehkä voisit kokeilla tällaista:

        $tieto1="";
        foreach($_POST as $tunnus => $arvo) {
        $tieto1 .= $tunnus . ": " . $arvo . "\n";
        }

        Ei aavistustakaan että toimiiko tuo, php kun on niin joustava kieli että en ole oppinut sen syntaksia vaikka kohta kymmenen vuotta olen sillä jo koodaillut...


      • apuatarttis
        kanssakoodari kirjoitti:

        Se piste tarkoittaa että merkkijonomuuttujaan sijoitettava arvo liitetään merkkijonon loppuun. Nyt kun poistit sen niin arvoista vain viimeinen jää muuttujaan.

        Virheilmoitus kertoi että tieto1-muuttujaa ei ole määritetty. Ehkä voisit kokeilla tällaista:

        $tieto1="";
        foreach($_POST as $tunnus => $arvo) {
        $tieto1 .= $tunnus . ": " . $arvo . "\n";
        }

        Ei aavistustakaan että toimiiko tuo, php kun on niin joustava kieli että en ole oppinut sen syntaksia vaikka kohta kymmenen vuotta olen sillä jo koodaillut...

        Juupelis juu eli määritin muuttujan tieto1. Nyt kun käytän sitä error reporting koodia siinä, niin ei enää ilmoita virhettä line 6:lla tai millään muullakaan.

        Kyseinen ongelma on siispä korjattu, MUTTA vaikka vanhoihin koodeihin lisäisin tuon tieto 1:n määrityksen, sähköpostilaatikkoni ei inahdakkaan. Toisaalta osasin jo arvatakkin, koska mikäli se tuon tieto 1:n määrittämisestä olisi kiinni, niin miksi se sitten toimi pitkään hyvin. Ja toiseksi vielä, että tieto 1 muuttujahan oikeasti periaatteessa on määritelty, koska siellä lomakkeen puolella on tieto1:n mukaan nimetty kenttä. Elikkä koodi poimii sen kentän tiedot arvoksi muuttujalle.

        Kiitos kuitenkin avusta (: josko jollakin vielä jotain ideoita löytyis?


      • kanssakoodari
        apuatarttis kirjoitti:

        Juupelis juu eli määritin muuttujan tieto1. Nyt kun käytän sitä error reporting koodia siinä, niin ei enää ilmoita virhettä line 6:lla tai millään muullakaan.

        Kyseinen ongelma on siispä korjattu, MUTTA vaikka vanhoihin koodeihin lisäisin tuon tieto 1:n määrityksen, sähköpostilaatikkoni ei inahdakkaan. Toisaalta osasin jo arvatakkin, koska mikäli se tuon tieto 1:n määrittämisestä olisi kiinni, niin miksi se sitten toimi pitkään hyvin. Ja toiseksi vielä, että tieto 1 muuttujahan oikeasti periaatteessa on määritelty, koska siellä lomakkeen puolella on tieto1:n mukaan nimetty kenttä. Elikkä koodi poimii sen kentän tiedot arvoksi muuttujalle.

        Kiitos kuitenkin avusta (: josko jollakin vielä jotain ideoita löytyis?

        mail-funktion manuaali (http://php.net/manual/en/function.mail.php) tiesi kertoa että lähettäjä pitäisi aina lisätä headereihin. Itse olen käyttänyt kyseistä funktiota tähän tapaan:

        $headers = 'From: lahettaja';
        mail($to, $subject, $message, $headers);

        Se että koodisi on toiminut aiemmin mutta yhtäkkiä lakkasi toimimasta viittaisi siihen että jokin on palvelimella muuttunut, ehkä php:stä on asennettu uusi versio tai jokin asetus (esimerkiksi juuri tuon From-kentän oletusarvo) on muuttunut. Aika vaikea sanoa että missä menee vikaan jos virheilmoitustakaan ei tule.

        Toivottavasti asia selviää.


      • apuatarttis
        kanssakoodari kirjoitti:

        mail-funktion manuaali (http://php.net/manual/en/function.mail.php) tiesi kertoa että lähettäjä pitäisi aina lisätä headereihin. Itse olen käyttänyt kyseistä funktiota tähän tapaan:

        $headers = 'From: lahettaja';
        mail($to, $subject, $message, $headers);

        Se että koodisi on toiminut aiemmin mutta yhtäkkiä lakkasi toimimasta viittaisi siihen että jokin on palvelimella muuttunut, ehkä php:stä on asennettu uusi versio tai jokin asetus (esimerkiksi juuri tuon From-kentän oletusarvo) on muuttunut. Aika vaikea sanoa että missä menee vikaan jos virheilmoitustakaan ei tule.

        Toivottavasti asia selviää.

        Kokeilin vielä tuota headerijuttua, eipä toimi edelleenkään. Lähiaikoina pitäisi tulla tunnukset uudelle palvelimelle, joten kokeilen vielä siellä, että jos toimisi.

        Kiitos kaikille kuitenkin avusta, ja jos jollekkin vielä jotain mieleen juolahtaa, niin ilmotelkaa ihmeessä (:


    • apuatarttis

      Koodi täysin toimiva, vika oli palvelimen puolella.
      Vaihdoin uuteen palvelimeen ja nyt toimii ja pelittää systeemit taas kuten aina ennekin.

      Vanha palvelin oli 000webhost.com sivustolta napattu.
      Jos jollekkin siis tulee samanlainen tilanne vastaan ja käytössä on 000webhost, älkää ihmetelkö.

    Ketjusta on poistettu 0 sääntöjenvastaista viestiä.

    Luetuimmat keskustelut

    1. Orpo hiiri kadoksissa, Marin jo kommentoi

      Kuinka on valtiojohto hukassa, kun vihollinen Grönlantia valloittaa? Putinisti Purra myös hiljaa kuin kusi sukassa.
      Maailman menoa
      117
      6319
    2. Lopeta jo pelleily, tiedän kyllä mitä yrität mies

      Et tule siinä onnistumaan. Tiedät kyllä, että tämä on just sulle. Sä et tule multa samaan minkäänlaista responssia, kosk
      Ikävä
      379
      6135
    3. Nuori lapualainen nainen tapettu Tampereella?

      Työ­matkalainen havahtui erikoiseen näkyyn hotellin käytävällä Tampereella – tämä kaikki epäillystä hotelli­surmasta tie
      Lapua
      69
      5930
    4. Tampereen "empatiatalu" - "Harvoin näkee mitään näin kajahtanutta"

      sanoo kokoomuslainen. Tampereen kaupunginvaltuuston maanantain kokouksessa käsiteltävä Tampereen uusi hyvinvointisuunni
      Maailman menoa
      344
      3962
    5. Lidl teki sen mistä puhuin jo vuosikymmen sitten

      Eli asiakkaat saavat nyt "skannata" ostoksensa keräilyvaiheessa omalla älypuhelimellaan, jolloin ei tarvitse mitään eril
      Maailman menoa
      145
      2365
    6. Ukraina, unohtui korona - Grönlanti, unohtu Ukraina

      Vinot silmät, unohtui Suomen valtiontalouden turmeleminen.
      Maailman menoa
      4
      2345
    7. Orpo pihalla kuin lumiukko

      Onneksi pääministerimme ei ole ulkopolitiikassa päättäjiemme kärki. Hänellä on täysin lapsellisia luuloja Trumpin ja USA
      Kansallinen Kokoomus
      119
      1403
    8. Onko täällä helmessä tapahtunut vakava rikos?

      Onko kuullut kukaan mitään.
      Haapavesi
      12
      1231
    9. Miten kauan sulla menisi

      Jos tulisit mun luo tänne nyt kahvinkeittoon?
      Ikävä
      185
      1077
    10. Miksi me oikein

      Rakastuttiin?
      Ikävä
      59
      887
    Aihe