Probleem na upgrade PHP 5.6 naar 7.2
-
Deze website heb ik in een ver verleden overgenomen van iemand die er mee gestopt was.
Ik ben helaas geen .php held dus zodra ik de upgrade doe van 5.6 naar 7.2 dan loopt het mail-formulier vast en krijg ik geen bevestiging meer te zien, na versturen. Met 5.6 is dit wel zo en geen probleem.
Wie kan mij helpen en mij vertellen welke aanpassing ik moet doen om het in PHP 7.2 werkend te krijgen.
Ik zal dit enorm waarderen! :-)
Ik zal het php.formulier ook hier neerzetten. Ik had ook de foutmelding-optie aangezet via one.com en toen kreeg ik deze melding te zien:
Fatal error: Uncaught Error: Call to undefined function eregi() in /customers/e/7/3/athenewateringen.nl/httpd.www/contact.php:251 Stack trace: #0 /customers/e/7/3/athenewateringen.nl/httpd.www/contact.php(175): afhandeling() #1 {main} thrown in /customers/e/7/3/athenewateringen.nl/httpd.www/contact.php on line 251
Alvast bedankt en wie o wie wilt helpen :)
Groet, Rich
PHPForm:
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
<?php
function toonpageheader()
{
echo "
<!-- Main -->
<div id=\"main-content\">
<article>
<h2 class=\"title-heading\">Athene Palace Wateringen Locatie</h2>
<p><h8>Hieronder ziet u ons Athene Palace Wateringen online reserverings-formulier.<br>
Daaronder ziet u ons contact-formulier, en daaronder 1 kaartje van google-maps inclusief routebeschrijving.</h8></p>
<p>
</article>
<!-- separator -->
<hr class=\"separator\">
<div id=\"holder\">
<div id=\"rm_reservation\" class=\"section-content\" align=\"center\">
<div class=\"rm_reservation_top\" style=\"text-align: center; \"></div>
<div class=\"rm_reservation_body\">
<p style=\"text-align: center; \"><!--style from SeatMe-->
<link type=\"text/css\" rel=\"stylesheet\" href=\"http://www.seatme.nl/css/srm.css\" /></p>
<div id=\"rm_wrapper\"><!-- this is the script which generates the form -->
<script language=\"javascript\" type=\"text/javascript\" src=\"http://www.seatme.nl/srm/?rid=4918&af=2832&lang=nl\"></script> <noscript> <div id=\"rm_form_wrapper\"> <h1 class=\"rm_header\"></h1> <div class=\"rm_box_wrapper\"> <div class=\"rm_gray_box\"> <a class=\"rm_page\" href=\"http://www.seatme.nl/restaurant/detail.aspx?rid=4918\"> Maak een reservering bij </a> </div> </div> </div> <div id=\"rm_logo_wrapper\"> <span class=\"rm_powered\">powered by</span> <a alt=\"Powered by SeatMe\" class=\"rm_logo\" href=\"http://www.seatme.nl\"></a> </div> </noscript></div>
</div>
<div class=\"rm_reservation_bottom\" style=\"text-align: center; \"></div>
</div>
</div>
";
}
if (!isset($_POST['zendform']))
{
toonpageheader();
toonformheader();
toonform();
toonpagefooter();
} else
{
afhandeling();
}
function toonformheader()
{
echo "
<section id=\"contact-form\">
<h2 class=\"form-heading\">Neemt u hier contact op</h2>
<p></p>";
}
function toonform()
{
echo "
<form class=\"contact-form clearfix\" action=\"contact.php\" method=\"post\">
<p class=\"adjust\">
<input type=\"hidden\" name=\"zendform\" value=\"1\">
<label for=\"name\">Naam <span>*</span> </label>
<input type=\"text\" name=\"name\" id=\"name\" value=\"\">
</p>
<p>
<label for=\"pn\">Telefoonnummer </label>
<input type=\"text\" name=\"pn\" id=\"pn\" value=\"\">
</p>
<p class=\"adjust\">
<label for=\"email\">E-mail <span>*</span></label>
<input type=\"text\" name=\"email\" id=\"email\" value=\"\">
</p>
<p>
<label for=\"reason\">Onderwerp </label>
<input type=\"text\" name=\"reason\" id=\"reason\" value=\"\">
</p>
<label for=\"message\">Bericht <span>*</span> </label>
<textarea name=\"message\" id=\"message\"></textarea>
<input type=\"submit\" name=\"submit\" value=\"Zend Bericht\">
</form>";
}
function afhandeling()
{
$email_ontvanger = "[email protected]";
$email_ontvanger2 = "";
$naam = $_POST['name'];
$tel = $_POST['pn'];
$email_verzender = $_POST['email'];
$header="From: $naam <$email_verzender>";
$onderwerp = $_POST['reason'];
$bericht = $_POST['message'];
// Controle op eventuele fouten
$fout = FALSE;
$naam_fout = FALSE;
$tel_fout = FALSE;
$email_fout = FALSE;
$onderwerp_fout = FALSE;
$bericht_fout = FALSE;
$patroon_naam = ".{2,}";
$patroon_email = ".+@.+\..+";
$patroon_onderwerp = ".{2,}";
$patroon_bericht = ".{2,}";
if (!eregi($patroon_naam,$naam))
{
$fout=TRUE;
$naam_fout=TRUE;
}
if (!eregi($patroon_email,$email_verzender))
{
$fout=TRUE;
$email_fout=TRUE;
}
if (!eregi($patroon_onderwerp,$onderwerp))
{
$fout=TRUE;
$onderwerp_fout=TRUE;
}
if (!eregi($patroon_bericht,$bericht))
{
$fout=TRUE;
$bericht_fout=TRUE;
}
if ($naam == "")
{
$fout=TRUE;
$naam_fout=TRUE;
}
if ($tel == "")
{
$fout=TRUE;
$tel_fout=TRUE;
}
if ($email_verzender == "")
{
$fout=TRUE;
$email_fout=TRUE;
}
if ($onderwerp == "")
{
$fout=TRUE;
$onderwerp_fout=TRUE;
}
if ($bericht == "")
{
$fout=TRUE;
$bericht_fout=TRUE;
}
// Controle of er fouten zijn gevonden
if ($fout)
{
// Als er fouten zijn gevonden dan komt hier het foutenlijstje
toonformheader();
echo "<table border=0 width=270 cellspacing=0 cellpadding=0><tr><td width=100%>";
echo "<font color=#FF0000>De volgende velden zijn niet volledig ingevuld:<br><br>";
echo ($naam_fout?"<font color=#FF0000>-Naam<br></font>":"");
echo ($tel_fout?"<font color=#FF0000>-Telefoonnummer<br></font>":"");
echo ($email_fout?"<font color=#FF0000>-Email<br></font>":"");
echo ($onderwerp_fout?"<font color=#FF0000>-Onderwerp<br></font>":"");
echo ($bericht_fout?"<font color=#FF0000>-Bericht<br></font>":"");
echo "</td></tr><tr><td><hr align=left color=#C0C0C0 width=90% noshade size=1></td></tr></table>";
echo "</font>";
toonform();
}
else
{
//e-mail bericht samenstellen
$emailbody = "
Naam: $naam
Tel: $tel
Email: $email_verzender
Onderwerp: $onderwerp
Bericht: $bericht
";
//e-mail bericht versturen
mail($email_ontvanger, $onderwerp, $emailbody, $header);
echo "<h7>Hartelijk dank. Uw e-mail is verstuurd!</h7>";
}
}
function toonpagefooter()
{
echo "
<!-- <div class=\"error-container\"></div>
<p id=\"message-sent\"><!-- Thanks for Your Message! <br /> We will get in touch shortly.--></p>
<br>
<iframe width=\"245\" height=\"289\" frameborder=\"1\" scrolling=\"no\" marginheight=\"0\" marginwidth=\"0\" src=\"https://maps.google.nl/maps?client=firefox-a&channel=sb&q=Herenstraat+88a+wateringen&ie=UTF8&hq=&hnear=Herenstraat+36,+2291+BH+Wateringen&gl=nl&t=m&z=14&ll=52.024156,4.276544&output=embed\"></iframe><br /><small><a href=\"https://maps.google.nl/maps?client=firefox-a&channel=sb&q=Herenstraat+88a+wateringen&ie=UTF8&hq=&hnear=Herenstraat+88a,+2291+BH+Wateringen&gl=nl&t=m&z=14&ll=52.024156,4.276544&source=embed\" style=\"color:#0000FF;text-align:left\">Grotere kaart weergeven</a></small>
<p>
</section>
</div><!-- End Main -->";
}
?>
function toonpageheader()
{
echo "
<!-- Main -->
<div id=\"main-content\">
<article>
<h2 class=\"title-heading\">Athene Palace Wateringen Locatie</h2>
<p><h8>Hieronder ziet u ons Athene Palace Wateringen online reserverings-formulier.<br>
Daaronder ziet u ons contact-formulier, en daaronder 1 kaartje van google-maps inclusief routebeschrijving.</h8></p>
<p>
</article>
<!-- separator -->
<hr class=\"separator\">
<div id=\"holder\">
<div id=\"rm_reservation\" class=\"section-content\" align=\"center\">
<div class=\"rm_reservation_top\" style=\"text-align: center; \"></div>
<div class=\"rm_reservation_body\">
<p style=\"text-align: center; \"><!--style from SeatMe-->
<link type=\"text/css\" rel=\"stylesheet\" href=\"http://www.seatme.nl/css/srm.css\" /></p>
<div id=\"rm_wrapper\"><!-- this is the script which generates the form -->
<script language=\"javascript\" type=\"text/javascript\" src=\"http://www.seatme.nl/srm/?rid=4918&af=2832&lang=nl\"></script> <noscript> <div id=\"rm_form_wrapper\"> <h1 class=\"rm_header\"></h1> <div class=\"rm_box_wrapper\"> <div class=\"rm_gray_box\"> <a class=\"rm_page\" href=\"http://www.seatme.nl/restaurant/detail.aspx?rid=4918\"> Maak een reservering bij </a> </div> </div> </div> <div id=\"rm_logo_wrapper\"> <span class=\"rm_powered\">powered by</span> <a alt=\"Powered by SeatMe\" class=\"rm_logo\" href=\"http://www.seatme.nl\"></a> </div> </noscript></div>
</div>
<div class=\"rm_reservation_bottom\" style=\"text-align: center; \"></div>
</div>
</div>
";
}
if (!isset($_POST['zendform']))
{
toonpageheader();
toonformheader();
toonform();
toonpagefooter();
} else
{
afhandeling();
}
function toonformheader()
{
echo "
<section id=\"contact-form\">
<h2 class=\"form-heading\">Neemt u hier contact op</h2>
<p></p>";
}
function toonform()
{
echo "
<form class=\"contact-form clearfix\" action=\"contact.php\" method=\"post\">
<p class=\"adjust\">
<input type=\"hidden\" name=\"zendform\" value=\"1\">
<label for=\"name\">Naam <span>*</span> </label>
<input type=\"text\" name=\"name\" id=\"name\" value=\"\">
</p>
<p>
<label for=\"pn\">Telefoonnummer </label>
<input type=\"text\" name=\"pn\" id=\"pn\" value=\"\">
</p>
<p class=\"adjust\">
<label for=\"email\">E-mail <span>*</span></label>
<input type=\"text\" name=\"email\" id=\"email\" value=\"\">
</p>
<p>
<label for=\"reason\">Onderwerp </label>
<input type=\"text\" name=\"reason\" id=\"reason\" value=\"\">
</p>
<label for=\"message\">Bericht <span>*</span> </label>
<textarea name=\"message\" id=\"message\"></textarea>
<input type=\"submit\" name=\"submit\" value=\"Zend Bericht\">
</form>";
}
function afhandeling()
{
$email_ontvanger = "[email protected]";
$email_ontvanger2 = "";
$naam = $_POST['name'];
$tel = $_POST['pn'];
$email_verzender = $_POST['email'];
$header="From: $naam <$email_verzender>";
$onderwerp = $_POST['reason'];
$bericht = $_POST['message'];
// Controle op eventuele fouten
$fout = FALSE;
$naam_fout = FALSE;
$tel_fout = FALSE;
$email_fout = FALSE;
$onderwerp_fout = FALSE;
$bericht_fout = FALSE;
$patroon_naam = ".{2,}";
$patroon_email = ".+@.+\..+";
$patroon_onderwerp = ".{2,}";
$patroon_bericht = ".{2,}";
if (!eregi($patroon_naam,$naam))
{
$fout=TRUE;
$naam_fout=TRUE;
}
if (!eregi($patroon_email,$email_verzender))
{
$fout=TRUE;
$email_fout=TRUE;
}
if (!eregi($patroon_onderwerp,$onderwerp))
{
$fout=TRUE;
$onderwerp_fout=TRUE;
}
if (!eregi($patroon_bericht,$bericht))
{
$fout=TRUE;
$bericht_fout=TRUE;
}
if ($naam == "")
{
$fout=TRUE;
$naam_fout=TRUE;
}
if ($tel == "")
{
$fout=TRUE;
$tel_fout=TRUE;
}
if ($email_verzender == "")
{
$fout=TRUE;
$email_fout=TRUE;
}
if ($onderwerp == "")
{
$fout=TRUE;
$onderwerp_fout=TRUE;
}
if ($bericht == "")
{
$fout=TRUE;
$bericht_fout=TRUE;
}
// Controle of er fouten zijn gevonden
if ($fout)
{
// Als er fouten zijn gevonden dan komt hier het foutenlijstje
toonformheader();
echo "<table border=0 width=270 cellspacing=0 cellpadding=0><tr><td width=100%>";
echo "<font color=#FF0000>De volgende velden zijn niet volledig ingevuld:<br><br>";
echo ($naam_fout?"<font color=#FF0000>-Naam<br></font>":"");
echo ($tel_fout?"<font color=#FF0000>-Telefoonnummer<br></font>":"");
echo ($email_fout?"<font color=#FF0000>-Email<br></font>":"");
echo ($onderwerp_fout?"<font color=#FF0000>-Onderwerp<br></font>":"");
echo ($bericht_fout?"<font color=#FF0000>-Bericht<br></font>":"");
echo "</td></tr><tr><td><hr align=left color=#C0C0C0 width=90% noshade size=1></td></tr></table>";
echo "</font>";
toonform();
}
else
{
//e-mail bericht samenstellen
$emailbody = "
Naam: $naam
Tel: $tel
Email: $email_verzender
Onderwerp: $onderwerp
Bericht: $bericht
";
//e-mail bericht versturen
mail($email_ontvanger, $onderwerp, $emailbody, $header);
echo "<h7>Hartelijk dank. Uw e-mail is verstuurd!</h7>";
}
}
function toonpagefooter()
{
echo "
<!-- <div class=\"error-container\"></div>
<p id=\"message-sent\"><!-- Thanks for Your Message! <br /> We will get in touch shortly.--></p>
<br>
<iframe width=\"245\" height=\"289\" frameborder=\"1\" scrolling=\"no\" marginheight=\"0\" marginwidth=\"0\" src=\"https://maps.google.nl/maps?client=firefox-a&channel=sb&q=Herenstraat+88a+wateringen&ie=UTF8&hq=&hnear=Herenstraat+36,+2291+BH+Wateringen&gl=nl&t=m&z=14&ll=52.024156,4.276544&output=embed\"></iframe><br /><small><a href=\"https://maps.google.nl/maps?client=firefox-a&channel=sb&q=Herenstraat+88a+wateringen&ie=UTF8&hq=&hnear=Herenstraat+88a,+2291+BH+Wateringen&gl=nl&t=m&z=14&ll=52.024156,4.276544&source=embed\" style=\"color:#0000FF;text-align:left\">Grotere kaart weergeven</a></small>
<p>
</section>
</div><!-- End Main -->";
}
?>
Gewijzigd op 02/11/2018 12:42:07 door Richard vdBerg
preg_match. Deze werkt wel volgens reguliere expressies met een 'delimiter' vóór en na. Ikzelf gebruik vaak / als karakter.
Zou je dit voortaan kunnen beperken tot relevante code? 291 regels zijn echt onnodig (vooral als er een hoop enters tussen zitten), als het maar om een enkele functie gaat. Dan zien we liever 10 regels voor en na.
De eregi() functie van PHP bestaat sinds PHP 7 niet meer en wordt al sinds PHP 5.3 gelabeld als: 'afgekeurd' (deprecated). Een vervanger hiervoor is Zou je dit voortaan kunnen beperken tot relevante code? 291 regels zijn echt onnodig (vooral als er een hoop enters tussen zitten), als het maar om een enkele functie gaat. Dan zien we liever 10 regels voor en na.
Gewijzigd op 02/11/2018 12:37:35 door - Ariën -
Ik weet eigenlijk bijna niets van PHP. Om het nu werkend te krijgen?
Is dit veel werk/moeite?
Wat en hoe kan ik het doen om het werkend te krijgen? Stap voor stap? :)
Dank je wel mensen! :)
filter_var in plaats dat je zelf moeilijk gaat doen met controles.
Eerst de normale reguliere expressie voor de naam:
Voor e-mail adressen kan je prima deze functie gebruiken:
Eigenlijk is het vreemd dat je steeds bij de naam kijkt of er een karakter in voorkomt. Je kan toch prima kijken met een lege string of deze leeg is? Daarvoor hoef je geen 'dure' reguliere expressie te gebruiken.
Het is eenvoudig te doen, ik zou wel het controleren van je mail overlaten aan PHP zelf met Eerst de normale reguliere expressie voor de naam:
Code (php)
1
2
3
4
5
6
7
2
3
4
5
6
7
<?php // deze regel is enkel voor de kleurtjes in dit blok, niet overnemen
$patroon_naam = "/.{2,}/"; // / om het 'patroon geplaatst'
if (!preg_match($patroon_naam,$naam)) {
/*Hier de rest */
?>
$patroon_naam = "/.{2,}/"; // / om het 'patroon geplaatst'
if (!preg_match($patroon_naam,$naam)) {
/*Hier de rest */
?>
Voor e-mail adressen kan je prima deze functie gebruiken:
Code (php)
1
2
3
4
5
6
7
8
9
2
3
4
5
6
7
8
9
<?php
$email = "[email protected]";
if (filter_var($email, FILTER_VALIDATE_EMAIL)) {
echo "E-mailadres is geldig";
} else {
echo "E-mailadres is niet geldig";
}
?>
$email = "[email protected]";
if (filter_var($email, FILTER_VALIDATE_EMAIL)) {
echo "E-mailadres is geldig";
} else {
echo "E-mailadres is niet geldig";
}
?>
Eigenlijk is het vreemd dat je steeds bij de naam kijkt of er een karakter in voorkomt. Je kan toch prima kijken met een lege string of deze leeg is? Daarvoor hoef je geen 'dure' reguliere expressie te gebruiken.
Gewijzigd op 02/11/2018 13:01:06 door - Ariën -
het script is even nagekeken - contact.php -
U maakt gebruik van een functie eregi( en deze functie werd reeds met PHP 7 verwijdert en zal dus inderdaad niet werken met versie 7,2. U dient de code van dit script aan te passen zodat deze wel compatibel is.
Ter info...
Dat klopt!
Naast een "upgrade" om de site weer werkend te krijgen zou ik deze in zijn totaliteit eens onder de loep nemen, dat alles kan een stuk korter en vooral beter leesbaar.
Om een metafoor te gebruiken - voordat je je huis gaat opknappen en wat muren gaat sauzen is het wellicht interessant om te kijken of het fundament ondertussen niet op instorten staat.
Volgens mij is deze code behoorlijk oud. En ik geef toe dat het nodig op de schop kan.
Download link
https://we.tl/t-EVmlaindy8
kunnen jullie deze dan aanpassen? Is dit een optie/mogelijk? Wat jullie mij tot nu toe vertellen is voor mij een beetje chinees :)
Dank je wel :)
Rich
Op sich is een complete herbouw niet verplicht, maar het maakt het wel fijner om in te werken.
Maar als je er echt niet uit komt dan kan ik wel even kijken of ik de code kan aanpassen.
Gewijzigd op 02/11/2018 13:16:50 door - Ariën -
Ik zal als dank even je webpagina promoten op mijn facebook kanalen en twitter, bij elkaar een paar duizend man!
Maar moet het eerst werken natuurlijk :)
thanks!
Ik ga vanmiddag even kijken, ik PM je wel.
Graag een onderbouwing erbij, zodat ik er hopelijk wat van kan leren :)
Nogmaals thanks!
Tevens heb ik hier en daar wat commentaar geplaatst met wat uitleg.
Gewijzigd op 04/11/2018 14:19:11 door - Ariën -