modx evo обновление с версии 1.0.15 к 1.2.1 и shopkeeper 1.3.4 на shopkeeper 1.3.6

Версия MODX: Evolution

Объясните пожалуйста что нужно менять.
Обновил modx evo 1.0.15 на версию 1.2.1
Так же обновил все плагины. включая ajax search, и теперь у меня верстка полетела Мне нужно лезть в чанки по выводу ajax search? что бы подправить все это дело.

после обновления пришлось устанавливать shopkeeper 1.3.6 Так как сам shopkeeper 1.3.4 не захотел обновляться.
Теперь при переходе к ресурсу выдает ошибки.

Trying to get property of non-object
« MODX Parse Error »
Error : mysql_real_escape_string(): Нет такого файла или каталога
Error information       
ErrorType[num]  WARNING[2]
File    /home/u381807/reon.ru/www/assets/snippets/shopkeeper/filter/form.php
Line    20
Source  $_REQUEST[$k[$i]] = mysql_real_escape_string(htmlspecialchars(strip_tags(trim($_REQUEST[$k[$i]]))));
Current Snippet filter-form
Current Plugin  phx(OnParseDocument)

Basic info      
REQUEST_URI     http://reon.ru/catalog/1240/1259/1261
Resource        [1261] РСТ-82АВ
Referer http://reon.ru/

Как мне теперь настроить shopkeeper 1.3.6?
это мне нужно в нутрь шаблона залезать и искать чанки по выводу шопкипера?

Егор
19.07.17 в 20:07

Комментарии (3)

Алексей Либер 20.07.2017 03:38 #

По AS — видимо при создании сайта вносились изменения в стандартные чанки, которые затерлись при обновлении. Да, вам нужно поправить соответствующие из них (лежат в папке /assets/snippets/ajaxSearch/templates).
На счет ШК — нет, проблема не в нем, а в кастомном дополнении filter.
Необходимо все прямые функции mysql заменить на modx dbapi. Например mysql_real_escape_string(); на $modx->db->escape();
Если приведете листинг файла — /assets/snippets/shopkeeper/filter/form.php то смогу подсказать что и на что нужно заменить.

Комментарий был изменён 20.07.2017 в 03:39

Егор 20.07.2017 11:13 #

<?php

// Show form
$show = true;
if(empty($_REQUEST['cat']) && empty($_REQUEST['etype']) && empty($_REQUEST['series'])){
	$show = false;
	// Populate request with folders
	$req = explode('/',$_SERVER['REQUEST_URI']);
	$ps = array(2=>'cat'/*,3=>'etype',4=>'series'*/);
	foreach ($ps as $k=>$param){
		$v = explode('?',$req[$k]);
		if($v[0] > 0)
			$_REQUEST[$param] = $v[0];
	}
}

$s = sizeof($_REQUEST);
$k = array_keys($_REQUEST);
for ($i=0; $i < $s; $i++) {
	$_REQUEST[$k[$i]] = mysql_real_escape_string(htmlspecialchars(strip_tags(trim($_REQUEST[$k[$i]]))));
}


// Reset form if category has changed
if($_SESSION['fcat']>0 && $_SESSION['fcat'] !== $_REQUEST['cat'])
	$_REQUEST = array('cat'=>$_REQUEST['cat']);
$_SESSION['fcat'] = intval($_REQUEST['cat']) > 0 ? $_REQUEST['cat'] : 0;

// Reset form if subcategory has changed
if($_SESSION['fetype']>0 && $_SESSION['fetype'] !== $_REQUEST['etype'])
	$_REQUEST = array('cat'=>$_REQUEST['cat'],'etype'=>$_REQUEST['etype']);
$_SESSION['fetype'] = intval($_REQUEST['etype']) > 0 ? $_REQUEST['etype'] : 0;


function formSelect($name,$options,$d='',$s='',$class='')
{
	$d = $d ? 'disabled="disabled"' : '';
	$no = !$s ? ' selected="selected"' : '';
	$class = $class ? ' '.$class : '';
	$out = '<label for="'.$name.'">'.$GLOBALS['params'][$name].'</label><select '.$d.' class="input-xlarge'.$class.'" id="'.$name.'" name="'.$name.'" autocomplete="off">
	<option value=""'.$no.'>Не выбрано</option>';
	if(sizeof($options)>0){
		foreach ($options as $id => $val) {
			$sel = $s==$id ? ' selected="selected"' : '';
			$out .= '<option value="'.$id.'"'.$sel.'>'.$val.'</option>';
		}
	}
	$out .= '</select>';
	return $out;
}

include $modx->config['base_path'] . 'assets/snippets/shopkeeper/filter/fieldmap.inc.php';
$GLOBALS['params'] = $fieldmap;
$GLOBALS['params']['cur'] .= ', А';
$GLOBALS['params']['volt'] .= ', В';

$fields = array();
$selects = array();
$last = '';
foreach ($GLOBALS['params'] as $name=>$label)
{
	switch ($name) {
		case 'cat':
			$_REQUEST[$name] = intval($_REQUEST[$name]);
			$id = $name=='cat' ? 6 : $_REQUEST[$name];

			// Get options
			$options = array();
			$vals = $modx->getDocumentChildren($id, 1, 0, 'id,pagetitle', 'isfolder=1', 'pagetitle', 'ASC');
			foreach ($vals as $v)
				$options[$v['id']] = $v['pagetitle'];

			// Form selects
			if(!empty($options)){
				$selects[$name] = formSelect($name,$options,false,$_REQUEST[$name],'ajax');
			} else {
				$selects[$name] = formSelect($name,array(),'disabled');
			}
			break;
		case 'etype':
		case 'series':
			$_REQUEST[$name] = intval($_REQUEST[$name]);
			if($_REQUEST[$name] > 0){
				$res = $modx->db->query('SELECT * FROM `modx_catalog_paramsvalues` WHERE `ID`='.$_REQUEST[$name].' ORDER BY `value` ASC');
				if($modx->db->getRecordCount($res) > 0)
				{
					$res = $modx->db->getRow($res);
					$lpt = $modx->db->query('SELECT * FROM `modx_site_content` WHERE `pagetitle`="'.$res['value'].'" AND `isfolder`=1');
					$roots = $out = array();
					while($lp = $modx->db->getRow($lpt))
						$roots[] = $lp['id'];

					if($name=='etype')
						foreach($roots as $id){
							$isroot = $modx->db->query('SELECT `id` FROM `modx_site_content` WHERE parent='.$id.' AND `isfolder`=1 ORDER BY `isfolder` DESC');
							if($modx->db->getRecordCount($isroot) > 0){
								$out[] = implode(',',$modx->getChildIds($id));
							} else {
								$out[] = $id;
							}
						}
					else
						$out = $roots;

					$last = implode(',',$out);
				}
			}
		default:
			$_REQUEST[$name] = intval($_REQUEST[$name]);
			if($_REQUEST[$name] > 0 && !in_array($name,array('etype','series')) )
				$fields[] = $_REQUEST[$name];
			
			$ids = $modx->db->query('SELECT `values` FROM `modx_catalog_prodvalues` WHERE `cat`="'.$_REQUEST['cat'].'" AND `param`="'.$name.'" LIMIT 1');
			if($modx->db->getRecordCount($ids) > 0)
			{
				if($_REQUEST[$name] > 0) $show = true;
				$options = array();
				$ids = $modx->db->getRow($ids);
				$ids['values'] = str_replace(',,', ",'',", $ids['values']);
				$q = $modx->db->query('SELECT * FROM `modx_catalog_paramsvalues` WHERE `ID` IN ('.$ids['values'].') ORDER BY `value` ASC');
				if($modx->db->getRecordCount($q) > 0)
					while($vals = $modx->db->getRow($q))
						$options[$vals['ID']] = $vals['value'];

				$selects[$name] = formSelect($name,$options,false,$_REQUEST[$name]);					
			} else {
				$selects[$name] = formSelect($name,array(),'disabled');
			}
	}
}

/* filter */
$filter = $show ? array('sc.isfolder !@eq 1') : array();
if(!empty($fields) && !empty($_GET) || !empty($last))
{
	$filter = $last ? array('sc.isfolder !@eq 1 AND sc.parent IN ('.$last.')') : array('sc.isfolder !@eq 1');
	if(sizeof($fields) > 0){
		$q = $modx->db->select('value','modx_catalog_paramsvalues','id IN ('.implode(',',$fields).')');
		if($modx->db->getRecordCount($q) > 0){
			$filter[1] = 'AND tvc.tmplvarid @eq "6"';
			while($param = $modx->db->getRow($q)){
				$filter[] =  'AND tvc.value LIKE "%=='.$param['value'].'%"';
			}
		}
	}
}
// echo'<pre>',print_r($filter),'</pre>';
if(!empty($filter)){
    // $modx->regClientStartupHTMLBlock('<link rel="canonical" href="'.$modx->config['site_url'].substr($modx->makeUrl($modx->documentIdentifier),1).'">');
    // set placeholder to get in catalogView
	$modx->setPlaceholder('where',implode(' ',$filter));
}
/* //filter */

$in = $show ? ' in' : '';
echo '<form action="'.$modx->makeUrl(3135).'" method="get" id="filter" class="margin graybox accordion">
        <h2 data-target="#flt" data-toggle="collapse" class="accordion-toggle">Поиск по товарам</h2>
        <div class="accordion-group">
            <div class="accordion-body row-fluid collapse'.$in.'" id="flt">
                <fieldset class="span4">
                    '.$selects['cat'].'
                    '.$selects['etype'].'
                    '.$selects['series'].'                    
                </fieldset>
                <fieldset class="span4">
                	'.$selects['ptype'].'
                    '.$selects['chrt'].'
                    '.$selects['chm'].'
                </fieldset>
                <fieldset class="span4">
                	'.$selects['atype'].'  
                    '.$selects['cur'].'                    
                    '.$selects['volt'].'
                </fieldset>                
                <div class="text-center">
                    <button class="btn btn-primary span4 up" type="submit">Искать по параметрам</button>
                </div>
            </div>
        </div>
        <div class="line">
            <a data-target="#flt" data-toggle="collapse" class="accordion-toggle arr'.$in.'"></a>
        </div>
        <div class="clearfix"></div>
        <script>
			$(function(){
				var filter = $("#filter");
				var ftoggle = filter.find(".accordion-toggle");
				$("#filter .accordion-group").on("hide show", function () {
			    	ftoggle.toggleClass("in");
			    });
				filter.on("change",".ajax",function(){
			    	$.post("/ajax",filter.serialize(),function(data){
			    		filter.replaceWith(data);
			    	});
			    }).on("submit",function(){
			    	$(this).find("select").each(function(){
			            if($(this).val() <= 0 )// || $(this).hasClass("ajax"))
			            	$(this).attr("name","");
			        });
			    });
			});
		</script>
    </form>';

Алексей Либер 23.07.2017 10:31 #
Извиняюсь за долгое ожидание.
Да, вам нужно поменять только эту строчку:
```
$_REQUEST[$k[$i]] = mysql_real_escape_string(htmlspecialchars(strip_tags(trim($_REQUEST[$k[$i]]))));
```
на эту
```
$_REQUEST[$k[$i]] =  $modx->db->escape(strip_tags(trim($_REQUEST[$k[$i]])));
```
Еще мне не нравится использование $_GLOBAL — но это уже отдельная песня

Для добавления комментариев вы должны авторизоваться или зарегистрироваться.