Předmět Autor Datum
Odstraníš to opravou skriptu, který odesílá maily. nový
host 12.02.2020 21:16
host
To chápu. Ale o jakou opravu by se mělo konkrétně jednat? <?php session_start(); /** * Front to the… nový
Magistr 12.02.2020 21:21
Magistr
Já tomu kódu moc nerozumím (a nejsem programátor), ale postupoval bych tak, že bych alespoň dočasně… nový
L-Core 13.02.2020 06:21
L-Core
on celý ten kód je prasečina. Od začátku do konce. nový
touchwood 13.02.2020 07:10
touchwood
máš tam 2x else. https://www.w3schools.com/php/php_if_else.asp edit: upravíš to tak, že první else… nový
touchwood 13.02.2020 07:05
touchwood
Nenapadlo tě, že celý ten skript se spouští dvakrát? nový
touchwood 13.02.2020 07:12
touchwood
No jo, ale proč. A lze to nějak odstranit? Magistr poslední
Magistr 13.02.2020 09:55
Magistr

To chápu. Ale o jakou opravu by se mělo konkrétně jednat?

<?php
session_start();
/**
* Front to the WordPress application. This file doesn't do anything, but
loads

* wp-blog-header.php which does and tells WordPress to load the theme.

*
* @package WordPress
*/

/**
* Tells WordPress to load the WordPress theme and output it.

*
* @var bool

*/
define('WP_USE_THEMES', true);
/** Loads the WordPress Environment and Template
*/
require( dirname( __FILE__ ) . '/wp-blog-header.php' );


$current_user = wp_get_current_user();
if($_SESSION["navstevaVRelaci"] != "ano" and $_SERVER["REMOTE_ADDR"] != "94.112.194.89" )
{
   if($current_user  -> display_name != "")
   {
      $hl = "From: PRIVÁTNÍ WEBOVÉ STRÁNKY ČIMELICE wordpress@cimelice1963.czweb.org";
      $hl .= "\nMIME-Version: 1.0\n";
      $hl .= "Content-Type: text/html;charset=\"UTF-8\"\n";
      if(strpos($current_user  -> display_name,"ová") === false and strpos($current_user  -> display_name,"ová") === false) {
mail("barveramartin@gmail.com", "Návštěvník", "Web právě navštívil " . $current_user  -> display_name,$hl);
} else {
mail("barveramartin@gmail.com", "Návštěvník", "Web právě navštívila " . $current_user  -> display_name,$hl);
} 
} else {
$hl = "From: PRIVÁTNÍ WEBOVÉ STRÁNKY ČIMELICE wordpress@cimelice1963.czweb.org";
$hl .= "\nMIME-Version: 1.0\n";
$hl .= "Content-Type: text/html;charset=\"UTF-8\"\n";
mail("barveramartin@gmail.com", "Návštěvník", "Web právě navštívil nepřihlášený uživatel. (Také je možné, že má zakázané cookies webu)",$hl);
$_SESSION["navstevaVRelaci"] != "ano" ; }}

?>

Děkuji.
Magistr

Dopr... chlape, to to nemůžeš dát do code tagu, aby se to dalo číst?! (touchwood)

Já tomu kódu moc nerozumím (a nejsem programátor), ale postupoval bych tak, že bych alespoň dočasně zrušil celé to if s rozlišením uživatele na -ová a ten zbytek. Co třeba paní Holá, Krejčí? Přece ta reakce může být třeba "Web právě navštívil/a", obzvláště dnes, když se už tak strikně nepřechyluje (Emma Smetana).
A ponechal bych místo if jen

mail("barveramartin@gmail.com", "Návštěvník", "Web právě navštívil/a " . $current_user -> display_name,$hl)

Proč tam máš v if 2x

strpos($current_user -> display_name,"ová") === false)

--------
Jestli je to co píšu nesmysl, někdo to smažte :)
Je to jen nápad laika.

máš tam 2x else.

https://www.w3schools.com/php/php_if_else.asp

edit: upravíš to tak, že první else předěláš na elseif a dáš tam podmínku takovou, aby splňovala negaci podmnínek u if (v podstatě stačí místo obou false dát true).
Jinak takto prasácky (a IMHO úplně zbytečně) psaný kód jsem už dlouho neviděl. To si opravdu necháváš posílat pokaždé mail, když se ti někdo přihlásí?

Zpět do poradny Odpovědět na původní otázku Nahoru