Forums / Developer / Change in advancedsearch.php

Change in advancedsearch.php

Author Message

Juan Pablo Vercesi

Thursday 09 February 2006 8:49:49 am

Hi there,

I'm working on the search engine of a site, and have found this difference between two versions of eZ that I can't figure out why.

Originally, this site was in eZ 3.3.4, whose <i>/kernel/content/advancedsearch.php</i> file has the following lines (near line 120):

$searchContentClassID = -1;
$searchContentClassAttributes = 0;
$searchContentClassAttributeArray = array();
if ( $http->hasVariable( 'SearchContentClassID' ) and
     $http->variable( 'SearchContentClassID' ) != -1 )
{
    $searchContentClassID = $http->variable( 'SearchContentClassID' );
    $searchContentClass =& eZContentClass::fetch( $searchContentClassID );
    $searchContentClassAttributeArray =& $searchContentClass->fetchSearchableAttributes();
}

On the other hand, ez 3.6.5 has these on the same place:

$searchContentClassID = -1;
$searchContentClassAttributes = 0;
$searchContentClassAttributeArray = array();
if ( $http->hasVariable( 'SearchContentClassID' ) and
     $http->variable( 'SearchContentClassID' ) != -1 and
     (int)$http->variable( 'SearchContentClassID' ) > 0 )
{
    $searchContentClassID = (int)$http->variable( 'SearchContentClassID' );
    $searchContentClass =& eZContentClass::fetch( $searchContentClassID );
    if ( is_object( $searchContentClass ) )
        $searchContentClassAttributeArray =& $searchContentClass->fetchSearchableAttributes();
}

As you can see, the line inside the <i>if</i> block that assigns a value to <i>$searchContentClassID</i> has a type cast. This causes that if <i>$http->variable( 'SearchContentClassID' )</i> were returning an array (which, in fact, is my case), it is converted to an int, and the returning value is always 1 (my guess is that that conversion returns a "true" value, which is that the array exists).

I know that I can solve my 'problem' just removing the type casting, but I post this because I'd like to know why the type cast was added, how it changes the search behaviour and anything that could be related to it.

Thanks

JP,
may the source be with you.

Marko Žmak

Friday 19 May 2006 5:40:00 am

I also noticed this and reported it as a bug:

http://ez.no/bugs/view/8312

and also suggested a fix.

--
Nothing is impossible. Not if you can imagine it!

Hubert Farnsworth

Nabil Alimi

Saturday 20 May 2006 5:52:55 am

Okkk my thread 1 line up there doesn't make sense anymore. :)

My blog : http://www.starnab.com/ezpublish / http://www.starnab.com/ / http://www.assiki-consulting.com
eZ Publish Freelance developper. Feel free to contact me +33 674 367 057
nabil at assiki d0t fr

eZ debug

Timing: Jan 18 2025 20:50:27
Script start
Timing: Jan 18 2025 20:50:27
Module start 'content'
Timing: Jan 18 2025 20:50:27
Module end 'content'
Timing: Jan 18 2025 20:50:27
Script end

Main resources:

Total runtime0.1912 sec
Peak memory usage2,048.0000 KB
Database Queries141

Timing points:

CheckpointStart (sec)Duration (sec)Memory at start (KB)Memory used (KB)
Script start 0.00000.0059 587.7109180.8359
Module start 'content' 0.00590.0058 768.546997.8984
Module end 'content' 0.01170.1794 866.4453528.2734
Script end 0.1911  1,394.7188 

Time accumulators:

 Accumulator Duration (sec) Duration (%) Count Average (sec)
Ini load
Load cache0.00392.0276200.0002
Check MTime0.00170.8778200.0001
Mysql Total
Database connection0.00060.306110.0006
Mysqli_queries0.145776.23671410.0010
Looping result0.00150.79071390.0000
Template Total0.178993.610.1789
Template load0.00070.391410.0007
Template processing0.178293.204510.1782
Override
Cache load0.00050.259410.0005
Sytem overhead
Fetch class attribute can translate value0.00080.411110.0008
XML
Image XML parsing0.00030.172610.0003
General
dbfile0.00261.3399200.0001
String conversion0.00000.003630.0000
Note: percentages do not add up to 100% because some accumulators overlap

CSS/JS files loaded with "ezjscPacker" during request:

CacheTypePacklevelSourceFiles
CSS0extension/community/design/community/stylesheets/ext/jquery.autocomplete.css
extension/community_design/design/suncana/stylesheets/scrollbars.css
extension/community_design/design/suncana/stylesheets/tabs.css
extension/community_design/design/suncana/stylesheets/roadmap.css
extension/community_design/design/suncana/stylesheets/content.css
extension/community_design/design/suncana/stylesheets/star-rating.css
extension/community_design/design/suncana/stylesheets/syntax_and_custom_tags.css
extension/community_design/design/suncana/stylesheets/buttons.css
extension/community_design/design/suncana/stylesheets/tweetbox.css
extension/community_design/design/suncana/stylesheets/jquery.fancybox-1.3.4.css
extension/bcsmoothgallery/design/standard/stylesheets/magnific-popup.css
extension/sevenx/design/simple/stylesheets/star_rating.css
extension/sevenx/design/simple/stylesheets/libs/fontawesome/css/all.min.css
extension/sevenx/design/simple/stylesheets/main.v02.css
extension/sevenx/design/simple/stylesheets/main.v02.res.css
JS0extension/ezjscore/design/standard/lib/yui/3.17.2/build/yui/yui-min.js
extension/ezjscore/design/standard/javascript/jquery-3.7.0.min.js
extension/community_design/design/suncana/javascript/jquery.ui.core.min.js
extension/community_design/design/suncana/javascript/jquery.ui.widget.min.js
extension/community_design/design/suncana/javascript/jquery.easing.1.3.js
extension/community_design/design/suncana/javascript/jquery.ui.tabs.js
extension/community_design/design/suncana/javascript/jquery.hoverIntent.min.js
extension/community_design/design/suncana/javascript/jquery.popmenu.js
extension/community_design/design/suncana/javascript/jScrollPane.js
extension/community_design/design/suncana/javascript/jquery.mousewheel.js
extension/community_design/design/suncana/javascript/jquery.cycle.all.js
extension/sevenx/design/simple/javascript/jquery.scrollTo.js
extension/community_design/design/suncana/javascript/jquery.cookie.js
extension/community_design/design/suncana/javascript/ezstarrating_jquery.js
extension/community_design/design/suncana/javascript/jquery.initboxes.js
extension/community_design/design/suncana/javascript/app.js
extension/community_design/design/suncana/javascript/twitterwidget.js
extension/community_design/design/suncana/javascript/community.js
extension/community_design/design/suncana/javascript/roadmap.js
extension/community_design/design/suncana/javascript/ez.js
extension/community_design/design/suncana/javascript/ezshareevents.js
extension/sevenx/design/simple/javascript/main.js

Templates used to render the page:

UsageRequested templateTemplateTemplate loadedEditOverride
1pagelayout.tpl<No override>extension/sevenx/design/simple/templates/pagelayout.tplEdit templateOverride template
 Number of times templates used: 1
 Number of unique templates used: 1

Time used to render debug report: 0.0003 secs