AndBicScadMedia / solr-php-client

Automatically exported from code.google.com/p/solr-php-client
Other
0 stars 0 forks source link

null vs. false fieldBoost #2

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
in Apache/Solr/Service.php

There are a couple places where we experienced problems because this
$fieldBoost variable: 

                if ($fieldBoost !== false)

is NULL, so it passed the test and then when sent to Solr causes failure. 
Either there is a problem when setting the values, or possibly this check
should be something more robust like:

if (is_numeric($fieldBoost))

Original issue reported on code.google.com by pwola...@gmail.com on 28 Jan 2009 at 6:23

GoogleCodeExporter commented 9 years ago
note - this problem occurs with multi-value fields, not single value

Original comment by pwola...@gmail.com on 28 Jan 2009 at 6:25

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
This diff shows our current work-around.  

Original comment by pwola...@gmail.com on 28 Jan 2009 at 6:38

Attachments:

GoogleCodeExporter commented 9 years ago
Can you see if my recent commit 
http://code.google.com/p/solr-php-client/source/detail?r=5 is adequate?

Original comment by donovan....@gmail.com on 29 Jan 2009 at 12:52

GoogleCodeExporter commented 9 years ago
It worked well for me.  I was able to index using it without error.

Original comment by jacobsi...@gmail.com on 29 Jan 2009 at 3:59

GoogleCodeExporter commented 9 years ago
r5 has been working fine for us, so I think this issue is fixed.

Original comment by pwola...@gmail.com on 9 Mar 2009 at 3:14

GoogleCodeExporter commented 9 years ago

Original comment by donovan....@gmail.com on 12 Mar 2009 at 3:50