sobi2 moduł all entries

[naszemiasto]

witam,

przerobiłem lekko moduł all entries w taki sposób że filtruje mi wszystkie wpisy w zależności od województwa. I jest ok. Potrzebuję jednak aby oprócz samej nazwy wpisu wyświetlało mi również miasto. I robi się niedobrze bo moduł przestał się wyświetlać. Mógłby ktoś zerknąć w kod i powiedzieć co zrobiłem źle ?
Strona stoi na joomla 1.5.22, wersja Sobi2 2.9.3.2




<?php
/**
* @version $Id: mod_sobi2allentries.php 4847 2009-01-23
17:42:35Z Sigrid Suski $
* @package: Sigsiu Online Business Index 2 All
entries Module
* ================================================== =
*
@author
* Name: Sigrid & Radek Suski, Sigsiu.NET
* Email: sobi@sigsiu.net
*
Url: http://www.sigsiu.net
*
================================================== =
* @copyright Copyright
(C) 2007-2009 Sigsiu.NET (http://www.sigsiu.net). All rights
reserved.
* @license see http://www.gnu.org/copyleft/gpl.html
GNU/GPL.
* SOBI2 Latest Module is free software; you can redistribute it
and/or modify
* it under the terms of the GNU General Public License as
published by
* the Free Software Foundation.
*/


(defined( '_VALID_MOS' ) || defined( '_JEXEC' ) ) || ( trigger_error(
'Restricted access', E_USER_ERROR ) && exit() );


defined( "DS" ) || define( "DS",DIRECTORY_SEPARATOR);


$add = defined( 'JPATH_SITE' ) ? DS.'mod_sobi2allentries' :
null;
defined( '_SOBI_CMSROOT' ) || define( '_SOBI_CMSROOT', str_replace(
DS.'modules'.$add, null, dirname( __FILE__ ) ) );
class_exists( 'sobi2Config'
) || require_once(
_SOBI_CMSROOT.DS.'components'.DS.'com_sobi2'.DS.'c onfig.class.php' );


//error_reporting(E_ALL);
//ini_set("display_errors","on");


//Get the parameters
$class = $params->get('moduleclass_sfx');


$table = $params->get('moduletable', 0); //only J1.0
$entryicon =
$params->get('entryicon',0);
$entryimage =
$params->get('entryimage',0);
$showTitle =
$params->get('title',1);
$maxlength = $params->get('item_length',
50);
$direction = $params->get('direction', 1);
$categorydepend =
$params->get('categorydepend', 0);
$catwoj =
$params->get('wojewodztwo', "");
if(!$limit || $limit <
1)
$limit = 5;


$config =& sobi2Config::getInstance();
$database =&
$config->getDb();
$S_Itemid = $config->sobi2Itemid;
$catId =
sobi2Config::request( $_REQUEST,'catid',0 );


$now = $config->getTimeAndDate();


$woj="(itemid IN (SELECT itemid FROM jos_sobi2_fields_data WHERE
fieldid=5 AND data_txt='{$catwoj}')) ";


if (($catId > 1) && ($categorydepend == 1)) {
$query =
"SELECT a.title, a.itemid, a.icon, a.image, b.data_txt FROM #__sobi2_item AS a
WHERE (itemid IN (SELECT itemid FROM #__sobi2_cat_items_relations WHERE catid =
{$catId})) LEFT JOIN #__sobi2_fields_data AS b ON b.itemid=a.itemid WHERE
b.fieldid=3 AND {$woj} ORDER BY title ASC ";
}
else
{
$query = "SELECT a.title, a.itemid, a.icon, a.image, b.data_txt
FROM #__sobi2_item AS a LEFT JOIN #__sobi2_fields_data AS b ON a.itemid=b.itemid
WHERE b.fieldid=3 AND {$woj} ORDER BY title ASC ";
}


$database->setQuery( $query );
$s_results =
$database->loadObjectList();


$iso = explode( '=', _ISO );
$encoding = strtoupper($iso[1]);


echo "\n";
echo "<!-- Start of SOBI2 All Entries Module
-->\n";
if( !( defined( "_JEXEC" ) ) ) {
if
($table)
echo "<table cellpadding=\"0\" cellspacing=\"0\"
class=\"moduletable{$class}\"><tr><td>\n";
}
if ($direction
== 1)
echo "<ul style=\"text-align:left;\"
class=\"sobi2latest{$class}\">\n";


if(count($s_results)) {
foreach($s_results as $s_result)
{
$url =
"index.php?option=com_sobi2&amp;sobi2Task=sobi2Details&amp; sobi2Id={$s_result->itemid}&amp;Itemid={$S_Itemid}";
$url
= sobi2Config::sef($url);


$myTitle =
$config->getSobiStr($s_result->title);
$myFullTitle =
$myTitle;
$myTown =
$config->getSobiStr($s_result->data_txt);
$town =
$myTown;


$icon = null;
if($entryicon)
{
if($s_result->icon &&
file_exists("{$config->absolutePath}/images/com_sobi2/clients/{$s_result->icon}"))
{
$icon = "<a href=\"{$url}\"
title=\"{$myFullTitle}\"><img style=\"border-style:none;\"
src=\"{$config->liveSite}/images/com_sobi2/clients/{$s_result->icon}\"
title=\"{$myFullTitle}\"
alt=\"{$myFullTitle}\"/></a>&nbsp;";
}
}
$image
= null;
if($entryimage)
{
if($s_result->image &&
file_exists("{$config->absolutePath}/images/com_sobi2/clients/{$s_result->image}"))
{
$image = "<a href=\"{$url}\"
title=\"{$myFullTitle}\"><img style=\"border-style:none;\"
src=\"{$config->liveSite}/images/com_sobi2/clients/{$s_result->image}\"
title=\"{$myFullTitle}\"
alt=\"{$myFullTitle}\"/></a>&nbsp;";
}
}
$e
= null;
if($showTitle) {
if
((function_exists("mb_strlen")) && (substr($encoding,0,3) == "UTF"))
{
$len = mb_strlen($myTitle,
$encoding);
}
else

$len =
strlen($myTitle);

if ($len >
$maxlength) {
if ((function_exists("mb_substr"))
&& (substr($encoding,0,3) == "UTF"))
{
$myTitle = mb_substr($myTitle,
0, $maxlength,
$encoding);
}
else

$myTitle = substr($myTitle, 0,
$maxlength);


$myTitle =
$myTitle."...";
}


$e
= "<a href=\"{$url}\" title=\"{$myFullTitle}\">{$myTitle}
{$town}</a>";
}
if ($icon || $image || $e)
{
if ($direction == 1)
echo
"<li
class=\"sobi2latest{$class}\">{$icon}{$image}{$e}</li>\n";
else
{
echo "<div class=\"sobi2latest{$class}\"
style=\"margin:
5px;\">{$icon}{$image}{$e}</div>\n";
}
}
}
}
if
($direction == 1)
echo "</ul>\n";
else
echo "<div
style=\"clear:both;\"></div>\n";

if( !( defined( "_JEXEC"
) ) ) {
if ($table)
echo
"</td></tr></table>\n";
}


echo "<!--End of SOBI2 All Entries Module -->\n";
?>

[Jola]

Witam,
na pierwszy rzut oka zamiast:

$woj="(itemid IN (SELECT itemid FROM jos_sobi2_fields_data WHERE fieldid=5 AND data_txt='{$catwoj}')) ";zapisz

$woj="(a.itemid IN (SELECT itemid FROM jos_sobi2_fields_data WHERE fieldid=5 AND data_txt='{$catwoj}')) ";Konstrukcja pytania do bazy jest wadliwa, powinno być:
SELECT ... FROM ... LEFT JOIN ON .... WHERE
Wszystkie pola w WHERE powinny mieć wskazanie na tabelę np. a.catid itd.

[naszemiasto]

Wielkie dzięki, zadziałało. Brakowało a przed itemid. Nie wziąłem tego pod uwagę bo to miało służyć do filtrowania wg. województwa - ofiara ze mnie :D

[naszemiasto]

pozwoliłem sobie odświeżyć ten temat w związku z kolejnym problemem.
Okazało się że muszę wyciągnąć z bazy również nazwę ulicy... i zrobił się problem.
Zmieniając wartość fieldid z 3 na 1 to działa, wyświetla nazwę, ale gdy chcę jednocześnie pobrać nazwę miasta i ulicę , to już nie.
Na czerwono zaznaczyłem co odpowiada za pobranie nazwy miasta ale nie wiem jak sformułować zapytanie o ulicę.
Próbowałem na kilka sposobów ale niestety nic nie zdziałałem
Może ktoś doradzić coś sensownego - jakąś drobną wskazówkę, bo nie mogę ruszyć z miejsca już drugi dzień.



<?php
/**
* @version $Id: mod_sobi2allentries.php 4847 2009-01-23
17:42:35Z Sigrid Suski $
* @package: Sigsiu Online Business Index 2 All
entries Module
* ================================================== =
*
@author
* Name: Sigrid & Radek Suski, Sigsiu.NET
* Email: sobi@sigsiu.net
*
Url: http://www.sigsiu.net
*
================================================== =
* @copyright Copyright
(C) 2007-2009 Sigsiu.NET (http://www.sigsiu.net). All rights
reserved.
* @license see http://www.gnu.org/copyleft/gpl.html
GNU/GPL.
* SOBI2 Latest Module is free software; you can redistribute it
and/or modify
* it under the terms of the GNU General Public License as
published by
* the Free Software Foundation.
*/

(defined( '_VALID_MOS' ) || defined( '_JEXEC' ) ) || ( trigger_error(
'Restricted access', E_USER_ERROR ) && exit() );

defined( "DS" ) || define( "DS",DIRECTORY_SEPARATOR);

$add = defined( 'JPATH_SITE' ) ? DS.'mod_sobi2allentries' :
null;
defined( '_SOBI_CMSROOT' ) || define( '_SOBI_CMSROOT', str_replace(
DS.'modules'.$add, null, dirname( __FILE__ ) ) );
class_exists( 'sobi2Config'
) || require_once(
_SOBI_CMSROOT.DS.'components'.DS.'com_sobi2'.DS.'c onfig.class.php' );

//error_reporting(E_ALL);
//ini_set("display_errors","on");

//Get the parameters
$class = $params->get('moduleclass_sfx');

$table = $params->get('moduletable', 0); //only J1.0
$entryicon = $params->get('entryicon',0);
$entryimage =$params->get('entryimage',0);
$showTitle = $params->get('title',1);
$maxlength = $params->get('item_length',50);
$direction = $params->get('direction', 1);
$categorydepend = $params->get('categorydepend', 0);
$catwoj = $params->get('wojewodztwo', "");
if(!$limit || $limit <1)
$limit = 5;

$config =& sobi2Config::getInstance();
$database =& $config->getDb();
$S_Itemid = $config->sobi2Itemid;
$catId = sobi2Config::request( $_REQUEST,'catid',0 );

$now = $config->getTimeAndDate();

$woj="(a.itemid IN (SELECT itemid FROM jos_sobi2_fields_data WHERE fieldid=5 AND data_txt='{$catwoj}')) ";

if (($catId > 1) && ($categorydepend == 1)) {
$query = "SELECT a.title, a.itemid, a.icon, a.image, b.data_txt FROM #__sobi2_item AS a WHERE (itemid IN (SELECT itemid FROM #__sobi2_cat_items_relations WHERE catid = {$catId})) LEFT JOIN #__sobi2_fields_data AS b ON b.itemid=a.itemid WHERE b.fieldid=3 AND {$woj} ORDER BY title ASC ";
}
else
{
$query = "SELECT a.title, a.itemid, a.icon, a.image, b.data_txt FROM #__sobi2_item AS a LEFT JOIN #__sobi2_fields_data AS b ON a.itemid=b.itemid WHERE b.fieldid=3 AND {$woj} ORDER BY title ASC ";
}

$database->setQuery( $query );
$s_results =$database->loadObjectList();

$iso = explode( '=', _ISO );
$encoding = strtoupper($iso[1]);

echo "\n";
echo "<!-- Start of SOBI2 All Entries Module-->\n";
if( !( defined( "_JEXEC" ) ) ) {
if
($table)
echo "<table cellpadding=\"0\" cellspacing=\"0\" class=\"moduletable{$class}\"><tr><td>\n";
}
if ($direction == 1)
echo "<ul style=\"text-align:left;\" class=\"sobi2latest{$class}\">\n";

if(count($s_results)) {
foreach($s_results as $s_result)
{
$url = "index.php?option=com_sobi2&amp;sobi2Task=sobi2Details&amp; sobi2Id={$s_result->itemid}&amp;Itemid={$S_Itemid}";
$url = sobi2Config::sef($url);

$myTitle = $config->getSobiStr($s_result->title);
$myFullTitle = $myTitle;
$myTown = $config->getSobiStr($s_result->data_txt);
$town = $myTown;

$icon = null;
if($entryicon)
{
if($s_result->icon && file_exists("{$config->absolutePath}/images/com_sobi2/clients/{$s_result->icon}"))
{
$icon = "<a href=\"{$url}\" title=\"{$myFullTitle}\"><img style=\"border-style:none;\" src=\"{$config->liveSite}/images/com_sobi2/clients/{$s_result->icon}\" title=\"{$myFullTitle}\" alt=\"{$myFullTitle}\"/></a>&nbsp;";
}
}
$image = null;
if($entryimage)
{
if($s_result->image && file_exists("{$config->absolutePath}/images/com_sobi2/clients/{$s_result->image}"))
{
$image = "<a href=\"{$url}\" title=\"{$myFullTitle}\"><img style=\"border-style:none;\" src=\"{$config->liveSite}/images/com_sobi2/clients/{$s_result->image}\" title=\"{$myFullTitle}\" alt=\"{$myFullTitle}\"/></a>&nbsp;";
}
}
$e = null;
if($showTitle) {
if
((function_exists("mb_strlen")) && (substr($encoding,0,3) == "UTF"))
{
$len = mb_strlen($myTitle,$encoding);
}
else
$len = strlen($myTitle);
if ($len >$maxlength) {
if ((function_exists("mb_substr")) && (substr($encoding,0,3) == "UTF"))
{
$myTitle = mb_substr($myTitle,0, $maxlength,$encoding);
}
else
$myTitle = substr($myTitle, 0, $maxlength);

$myTitle = $myTitle."...";
}

$e = "<table><tr><td width=150px><a href=\"{$url}\" title=\"{$myFullTitle}\">{$myTitle}</a></td><td><a href=\"{$url}\" title=\"{$myFullTitle}\">{$town}</a></td></tr></table>";
}
if ($icon || $image || $e)
{
if ($direction == 1)
echo
"<li class=\"sobi2latest{$class}\">{$icon}{$image}{$e}</li>\n";
else
{
echo "<div class=\"sobi2latest{$class}\" style=\"margin:5px;\">{$icon}{$image}{$e}</div>\n";
}
}
}
}else echo "Brak wpisów w tym województwie.";
if
($direction == 1)
echo "</ul>\n";
else
echo "<div style=\"clear:both;text-align:center;\"></div>\n";
if( !( defined( "_JEXEC"
) ) ) {
if ($table)
echo "</td></tr></table>\n";
}

echo "<!--End of SOBI2 All Entries Module -->\n";
?>



---------- Post dodany o 21:03 ---------- Poprzedni post był o 14:57 ----------

temat do zamknięcia, poradziłem sobie :)