[PHP]Naprawa błędu w dodawaniu strony, PHP, Smarty Opcje

[Maker5]

Witam

Mam problem z moim panelem CMS.

Błąd polega na tym, że w trakcie dodawania nowej podstrony na stronie internetowej poprzez panel CMS

i kliknięcia zapisz pojawia się biała strona zamiast dodawać nową podstronę na stronę internetową.

Panel oparty o PHP Smarty.

Poniżej screeny pokazujące o co chodzi.

http://prnt.sc/og82qt
http://prnt.sc/og830p
http://prnt.sc/og859i

Ten post edytował Maker5 17.07.2019, 11:45:53

[nospor]

Jedyne co ci mozemy powiedziec na podstawie takiej masie informacji to: przejrzyj logi

[Maker5]

Jedyne co ci mozemy powiedziec na podstawie takiej masie informacji to: przejrzyj logi

Jakie logi?
na serwerze czy w konsoli chrome?

[nospor]

A widzisz jakies interesujace info w logach consoli chrome?

Tak, na serwerze. Logi apache/nginx ewentualnie logi twojej aplikacji bo i ona moze jakies logi zapisywac

[Maker5]

A widzisz jakies interesujace info w logach consoli chrome?

Tak, na serwerze. Logi apache/nginx ewentualnie logi twojej aplikacji bo i ona moze jakies logi zapisywac

ok mam, miałeś rację
oto treść pliku log
adres zmieniłem bo mi sugerowano

[PHP] pobierz, plaintext 
/home/srw_strona/strona.pl/lib/classes/Session.class.php on line 29
[17-Jul-2019 13:55:37 Europe/Warsaw] PHP Fatal error:  Uncaught ArgumentCountError: Too few arguments to function valid_text(), 3 passed in /home/srw_strona/strona.pl/admin/action/content.module.php on line 485 and at least 4 expected in /home/srw_strona/strona.pl/lib/MVC.function.php:460
Stack trace:
#0 /home/srw_strona/strona.pl/admin/action/content.module.php(485): valid_text('text', 'Jakas tresc<br ...', Object(Form))
#1 /home/srw_strona/strona.pl/admin/index.php(109): contentMultiActionController->form_action()
#2 /home/srw_strona/strona.pl/admin/index.php(162): MVC_Controller->Process()
#3 {main}
  thrown in /home/srw_strona/strona.pl/lib/MVC.function.php on line 460
[PHP] pobierz, plaintext 

Ten post edytował Maker5 17.07.2019, 12:56:45

[nospor]

Too few arguments to function valid_text(), 3 passed in /home/srw_strona/strona.pl/admin/action/content.module.php on line 485 and at least 4 expected

No i wszystko masz jak na dloni
Teraz idziesz do content.module.php linia 485 i poprawiasz

[Maker5]

No i wszystko masz jak na dloni
Teraz idziesz do content.module.php linia 485 i poprawiasz

ale jak to poprawić, pomóż proszę bo nie wiem

załączam całą treść tego pliku content.module.php

linia 485 zaczyna się od formuły: $text = valid_text($field_2, get_http_request($field_2), $this->form);
jest na końcu poniżej

[PHP] pobierz, plaintext 
	public function form_action()
	{
		$field = "content_id";
		$content_id = get_http_request($field);
		$this->form->setValue($field, $content_id);
 
		$field = "subaction";
		$subaction = get_http_request($field);
		$this->form->setValue($field, $subaction);
 
		if($content_id != '' && eregi('^[0-9]+$', $content_id) && $this->content->check_id($content_id))
		{
			$field_1 = "title";
			$title = valid_input($field_1, get_http_request($field_1), $this->form, true);
 
			$field_2 = "text";
			$text = valid_text($field_2, get_http_request($field_2), $this->form);
 
			//$purifier = new HTMLPurifier();
      //$text = $purifier->purify($text);
[PHP] pobierz, plaintext 

[nospor]

No ja tez nie wiem.
Blad wyraznie mowi, ze funkcja valid_text() wymaga przynajmniej 4 parametrow, a w kodzie jest odpalana tylko z 3.

Nie aktualizowales ostatnio czegos tam w tej swojej appce?
Jak wyglada kod funkcji valid_text() ?

[Maker5]

No ja tez nie wiem.
Blad wyraznie mowi, ze funkcja valid_text() wymaga przynajmniej 4 parametrow, a w kodzie jest odpalana tylko z 3.

Nie aktualizowales ostatnio czegos tam w tej swojej appce?
Jak wyglada kod funkcji valid_text() ?

tu jest chyba valid text

[PHP] pobierz, plaintext 
function valid_text($field, $value, &$form, $required, $nl2br = false) {
    if (!$value || strlen($value = trim($value)) == 0) {
        if ($required == true)
            $form->setError($field, 'empty');
        else
            $value = '';
    } else {
        if (get_magic_quotes_gpc())
            $value = stripslashes($value);
        if ($nl2br) {
            $value = nl2br($value);
        }
        $form->setValue($field, $value);
        $value = addslashes($value);
    }
    return $value;
}
[PHP] pobierz, plaintext 

[nospor]

No to zmien
text = valid_text($field_2, get_http_request($field_2), $this->form);
na
text = valid_text($field_2, get_http_request($field_2), $this->form, true);
albo
text = valid_text($field_2, get_http_request($field_2), $this->form, false);

w zaleznosci czy wymagasz by pole bylo wymagane czy nie

[Maker5]

No to zmien
text = valid_text($field_2, get_http_request($field_2), $this->form);
na
text = valid_text($field_2, get_http_request($field_2), $this->form, true);
albo
text = valid_text($field_2, get_http_request($field_2), $this->form, false);

w zaleznosci czy wymagasz by pole bylo wymagane czy nie

Zmieniłem na true i działa dziękuję

sprawdzam jeszcze raz błędy w logach i mam kolejne dwa może wiesz czym spowodowane
poniżej szczegoły

[PHP] pobierz, plaintext 
[17-Jul-2019 16:43:05 Europe/Warsaw] PHP Warning:  session_regenerate_id(): Cannot regenerate session id - session is not active in /home/srw_strona/strona.pl/lib/classes/Session.class.php on line 29
[PHP] pobierz, plaintext 

zawartość session.class

[PHP] pobierz, plaintext 
<?php
 
class Session
{
  private static $instance;
 
  private function __construct() {
    $this->startSession();
  }
 
  public static function singleton() {
    if (!isset(self::$instance)) {
      $c = __CLASS__;
      self::$instance = new $c;
    }
    return self::$instance;
  }
 
  public function __clone() {
    trigger_error('Klonowanie zabronione.', E_USER_ERROR);
  }
 
  private function startSession() {
    $session_key = substr(md5(dirname(__FILE__)), 0, 8);
    session_name('SESSID' . $session_key);
 
    ini_set('session.use_trans_sid', 0);
 
    session_regenerate_id();   
    session_start();
 
  }
}
 
[PHP] pobierz, plaintext 

[nospor]

Komunikat jest dosc wyrazny...

zmien kolejnosc o tutaj
session_regenerate_id();
session_start();

[Maker5]

Komunikat jest dosc wyrazny...

zmien kolejnosc o tutaj
session_regenerate_id();
session_start();

dziękuję, nie pojawia się już w logach

i ostatni błąd jeśli mogę Cię prosić

[PHP] pobierz, plaintext 
[17-Jul-2019 16:58:52 Europe/Warsaw] PHP Warning:  preg_match(): Unknown modifier ']' in /home/srw_strona/strona.pl/lib/MVC.function.php on line 632
[PHP] pobierz, plaintext 

poniżej treść MVC.function.php - linia 632 zaczyna się od: return preg_match('/' . $pattern . '/', $value);

[PHP] pobierz, plaintext 
if (!function_exists('eregi')) {
 
    function eregi($pattern, $value, $output = false) {
        if (!$output) {
            return preg_match('/' . $pattern . '/', $value);
        } else {
            preg_match('/' . $pattern . '/', $value, $output);
            return $output;
        }
    }
 
}
[PHP] pobierz, plaintext 

[nospor]

Dobrzy by zobaczyc co zawiera zmienna $pattern. Da sie?

[Maker5]

Dobrzy by zobaczyc co zawiera zmienna $pattern. Da sie?

jesli to w tym samym pliku to kod poniżej

[PHP] pobierz, plaintext 
/**
 * Method to sanitize incoming html.
 * Take from cakephp (http://cakephp.org)
 * Licensed under the MIT License
 *
 * @param unknown_type $string
 * @param unknown_type $remove
 * @return unknown
 */
function clean_html($string, $remove = false) {
    if ($remove) {
        $string = strip_tags($string);
    } else {
        $patterns = array("/\&/", "/%/", "/</", "/>/", '/"/', "/'/", "/\(/", "/\)/", "/\+/", "/-/");
        $replacements = array("&amp;", "%", "&lt;", "&gt;", "&quot;", "'", "(", ")", "+", "-");
        $string = preg_replace($patterns, $replacements, $string);
    }
    return $string;
}
 
function seo_html($string) {
    $a = array('"', '\'');
    $b = array('', '');
    $string = str_replace($a, $b, $string);
    $string = preg_match("[^ĘÓĄŚŁŻŹĆŃęóąśłżźćńA-Za-z0-9]", " ", $string);
    $string = preg_replace("/\s+/", " ", $string);
    $string = preg_replace("/^\W+|\W+$/", "", $string);
    return $string;
}
 
// ================================================================================
=====================
// internal function for utf8 decoding
// thanks to Jamie Pratt for noticing that PHP's function is a little
// screwy
function my_utf8_decode($string) {
    return strtr($string, "?ĽľŔÁÂĂÄĹĆÇČÉĘËĚÍÎĎĐŃŇÓÔŐÖŘŮÚŰÜÝßŕáâăäĺćçčéęëěíîďđńňóôőöřůúűüý˙", "SOZsozYYuAAAAAAACEEEEIIIIDNOOOOOOUUUUYsaaaaaaaceeeeiiiionoooooouuuuyy");
}
 
// sanitize a string in prep for passing a single argument to system() (or similar)
// input string, returns string stripped of special characters
function sanitize_system_string($string) {
    $pattern = '/(;|\||`|>|<|&|^|"|' . "\n|\r|'" . '|{|}|[|]|\)|\()/i'; // no piping, passing possible environment variables ($),
    // seperate commands, nested execution, file redirection,
    // background processing, special commands (backspace, etc.), quotes
    // newlines, or some other special characters
    $string = preg_replace($pattern, '', $string);
    $string = '"' . preg_replace('/\$/', '\\\$', $string) . '"'; //make sure this is only interpretted as ONE argument
    return $string;
}
 
// sanitize a string for SQL input (simple slash out quotes and slashes)
// input string, returns string with slashed out quotes
function sanitize_sql_string($string) {
    $pattern[0] = '/(\\\\)/';
    $pattern[1] = "/\"/";
    $pattern[2] = "/'/";
    $replacement[0] = '\\\\\\';
    $replacement[1] = '\"';
    $replacement[2] = "\\'";
    return preg_replace($pattern, $replacement, $string);
}
 
// sanitize a string for HTML (make sure nothing gets interpretted!)
function sanitize_html_string($string) {
    $pattern[0] = '/\&/';
    $pattern[1] = '/</';
    $pattern[2] = "/>/";
    $pattern[3] = '/\n/';
    $pattern[4] = '/"/';
    $pattern[5] = "/'/";
    $pattern[6] = "/%/";
    $pattern[7] = '/\(/';
    $pattern[8] = '/\)/';
    $pattern[9] = '/\+/';
    $pattern[10] = '/-/';
    $replacement[0] = '&amp;';
    $replacement[1] = '&lt;';
    $replacement[2] = '&gt;';
    $replacement[3] = '<br>';
    $replacement[4] = '&quot;';
    $replacement[5] = ''';
    $replacement[6] = '%';
    $replacement[7] = '(';
    $replacement[8] = ')';
    $replacement[9] = '+';
    $replacement[10] = '-';
    return preg_replace($pattern, $replacement, $string);
}
 
// ================================================================================
=====================
[PHP] pobierz, plaintext 

[nospor]

czyli nie da sie....

strzelam:
Zamien
return preg_match('/' . $pattern . '/', $value);
na
return preg_match($pattern, $value);
a jak to nie pomoze to zamien na
return preg_match('#' . $pattern . '#', $value);

[Maker5]

czyli nie da sie....

strzelam:
Zamien
return preg_match('/' . $pattern . '/', $value);
na
return preg_match($pattern, $value);
a jak to nie pomoze to zamien na
return preg_match('#' . $pattern . '#', $value);

Wstawiłem ten drugi kod bo przy pierwszym pokazywało znów ten sam błąd z tą różnicą, że zamiast Unknown modifier ']' było Unknown modifier '('

ale teraz pojawia się inny w logu

[PHP] pobierz, plaintext 
[17-Jul-2019 17:33:54 Europe/Warsaw] PHP Warning:  array_pop() expects parameter 1 to be array, null given in /home/srw_strona/strona.pl/lib/classes/ScoutAgent.class.php on line 392
[PHP] pobierz, plaintext 

Zawartość ScoutAgent.class.php poniżej. Linia 392 zaczyna się od: $this->user_browser = array('name' => $browser[1], 'addon' => $addon, 'version' => array_pop($wersja), 'bot' => $browser[2]);

[PHP] pobierz, plaintext 
  /**
   * Browser.
   */
  private function _get_browser_addon()
  {
    while (list(,$addon) = each(parent::$tblb))
    {
 
     /* if (eregi($addon[0], $this->user_agent, $wersja))
      {
        return $addon[1];
      }
      * 
      */
 
    }
    return null;
  }
 
  private function _get_browser()
  {
    while (list(,$browser) = each (parent::$tblc))
    {
      if (eregi($browser[0], $this->user_agent, $wersja))
      {
        $addon = ($browser[1] == 'Internet Explorer') ? $this->_get_browser_addon() : null;
        if (!isset($browser[2]))
        {
          $browser[2] = false;
        }
        $this->user_browser = array('name' => $browser[1], 'addon' => $addon, 'version' => array_pop($wersja), 'bot' => $browser[2]);
        return;
      }
    }
    $this->user_browser = false;
  }
 
  public function is_browser()
  {
    if (!isset($this->user_browser))
    {
      $this->_get_browser();
    }
    return is_array($this->user_browser);
  }
 
  public function browser($case = '')
  {
    if (!$this->is_browser())
    {
      return;
    }
 
    switch (strtolower($case))
    {
      case 'name' : return $this->user_browser['name']; break;
      case 'addon' : return $this->user_browser['addon']; break;
      case 'version' : return $this->user_browser['version']; break;
      default :
        $out = $this->user_browser['name'].' '.$this->user_browser['version'];
        if (!empty($this->user_browser['addon']))
        {
          $out .= ' ('.$this->user_browser['addon'].')';
        }
        return $out;
        break;
    }
  }
 
  /**
   * System.
   */
[PHP] pobierz, plaintext 

[nospor]

Przed ta linia daj

if ($wersja === null) {
$wersja = [];
}

[Maker5]

Przed ta linia daj

if ($wersja === null) {
$wersja = [];
}

Dodałem to przed tą linią 392 i po kliknięciu dodaj stronę mam error 524 na cloudflare

[PHP] pobierz, plaintext 
 private function _get_browser()
  {
    while (list(,$browser) = each (parent::$tblc))
    {
      if (eregi($browser[0], $this->user_agent, $wersja))
      {
        $addon = ($browser[1] == 'Internet Explorer') ? $this->_get_browser_addon() : null;
        if (!isset($browser[2]))
        {
          $browser[2] = false;
        }
		  if ($wersja === null) {
$wersja = [];
}
        $this->user_browser = array('name' => $browser[1], 'addon' => $addon, 'version' => array_pop($wersja), 'bot' => $browser[2]);
        return;
      }
    }
    $this->user_browser = false;
  }
[PHP] pobierz, plaintext 

[nospor]

A jaka masz wersje php?

Zamien
$wersja = [];
na
$wersja = array();

[Maker5]

A jaka masz wersje php?

Zamien
$wersja = [];
na
$wersja = array();

Mam PHP 7.1 na hostingu

teraz działa
nie ma błędu w logach
dziękuję

a czy wiesz może dlaczego wczytywanie po kliknięciu "dodaj stronę" nie trwa powiedzmy do 5 sekund tylko z 30 sekund?
od czego to może zależeć?

[nospor]

Bo operacje jakies dziwne sa robione?
Patrzac jak kiepsko napisany jest ten skrypt nie zdziwilbym sie jakby byl totalnie nieoptymalny pod katem bazy rowniez

[Maker5]

Bo operacje jakies dziwne sa robione?
Patrzac jak kiepsko napisany jest ten skrypt nie zdziwilbym sie jakby byl totalnie nieoptymalny pod katem bazy rowniez

chciałbyś zająć się tym odpłatnie?