Weird bug with browse form version 4.7.1, please fix

Dear Flynax community,

I just found this weird bug and yes it effects me and I think many other who use browse form builder. Basically if you input any field into Browse Form Builder of the category, you will get one exact value for all listing show on listing list, even though we have different value from different listings. Here is how to reproduce it on flynax demo site

First go to admincp Browse Form Builder of Category Home here:
https://classifieds.demoflynax.com/a...orm=short_form
- Try to remove Property for, Add City

Then check here:
https://classifieds.demoflynax.com/u...es/houses.html
You will see all listing short/browse value as: Peoria
Attachment 3217

Try again, but instead of City, try "State/Province/Region"
You will get all all listing short/browse value as: Arizona
Attachment 3219

Try again with Garage
All No, when some of them are yes,
Attachment 3218

Please fix this.

Hello Wei,

We know about this bug and solution will be included in the next update

meanwhile, do the following to fix this problem:

go to ftp > includes > classes > rlListings.class.php > public function getListings > find /* replace hook */ (~728 row)

replace code below:

PHP Code:

---

Only registered members can view the code. 


---

with:

PHP Code:

---

Only registered members can view the code. 


---

We apologize for any inconvenience!

Hi Rudi,

I did the change but it didnt fix the problem, still the same California in all listing on my website. I think it was a problem in ftp > includes > classes > rlListing.class.php because I did experiment by copy old rlListing.class.php from version 4.6.2 to 4.7.1 and it shows correctly, I might be wrong. Anyway, I restore everything back to the way it was with the issue, and I created a ticket with my ftp info XJG-852954. Please take a look if you have the time. Thank you.

Quote:

---

Originally Posted by Rudi

Sorry, my bad!

I made a typo in class name

now it's correct

---

Hi Rudi,

I actually did replace your code in rlListing.class.php too earlier. But it has 2 problems:
1. The title of the listing in listing list return generic "listing"
2. If short form has category field, it will not show category. I have room for rent and I have it show category (room for rent) in short form and after replace code, the category no show in short form in listing list.

Other than these two issue, fields show correct as I get correct state and city now. May be another typo in your code?

There is no typo anymore

I think you should create a ticket to look into it

or one last try:
use my quick solution to this, if it doesn't help it means there is something wrong with the settings on your site

revert all changes and below /* replace hook */

find and replace:

PHP Code:

---

Only registered members can view the code. 


---

with:

PHP Code:

---

Only registered members can view the code. 


---

Better to use fix below, I've tested it again and it works

[QUOTE=Rudi;25868]There is no typo anymore

I think you should create a ticket to look into it

or one last try:
use my quick solution to this, if it doesn't help it means there is something wrong with the settings on your site

revert all changes and below /* replace hook */

find and replace:

PHP Code:

---

Only registered members can view the code. 


---

with:

PHP Code:

---

Only registered members can view the code. 


---

/QUOTE]

Hi Rudi,

I just revert everything to where it has problem and apply this simple change, just remove IF. And it worked. So no need for the change above
Thank you for your help :)))

Hello Wei, Can I get some clarification on this fix please ? you say you just removed the > if ? does that include the bottom close bracket ? example

Quote:

---

if (!$short_form_fields) {
$short_form_fields = $this->getFormFields(
$listing['Category_ID'],
'short_forms',
$listing['Listing_type'],
$listing['Parent_IDs']
);
}

---

to

Quote:

---

(!$short_form_fields) {
$short_form_fields = $this->getFormFields(
$listing['Category_ID'],
'short_forms',
$listing['Listing_type'],
$listing['Parent_IDs']
}

---

or do I have it wrong ?

Pete

Hi Pete,

Just delete the red part

Quote:

---

if (!$short_form_fields) {
$short_form_fields = $this->getFormFields(
$listing['Category_ID'],
'short_forms',
$listing['Listing_type'],
$listing['Parent_IDs']
);
}

---

It should be like this

Quote:

---

$short_form_fields = $this->getFormFields(
$listing['Category_ID'],
'short_forms',
$listing['Listing_type'],
$listing['Parent_IDs']
);

---

Ok Brilliant I have it, thanks

Guys,

You'd better use the final solution I mentioned below, it works properly

so I've posted here again:


go to ftp > includes > classes > rlListings.class.php > public function getListings > find /* replace hook */ (~728 row)

replace code below:

PHP Code:

---

Only registered members can view the code. 


---

with:

PHP Code:

---

Only registered members can view the code. 


---

Thanks Rudi, appreciated.

[QUOTE=Rudi;25936]Guys,

You'd better use the final solution I mentioned below, it works properly

so I've posted here again:


go to ftp > includes > classes > rlListings.class.php > public function getListings > find /* replace hook */ (~728 row)

replace code below:

PHP Code:

---

Only registered members can view the code. 


---

with:

PHP Code:

---

Only registered members can view the code. 


---

/QUOTE]

Thanks Rudi,

I just tried this solution instead and it does work.