Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Filter by Brand not working #107

Closed
Twilight-Computer opened this issue Mar 30, 2024 · 8 comments
Closed

Filter by Brand not working #107

Twilight-Computer opened this issue Mar 30, 2024 · 8 comments

Comments

@Twilight-Computer
Copy link

I am not sure if this filter is enabled.
On the Shop page - selecting any "Brand" does not filter the results.
image

@givanz
Copy link
Owner

givanz commented Mar 31, 2024

Filters component is missing only the html in theme is added, I will add it in the next commit.

@givanz
Copy link
Owner

givanz commented Mar 31, 2024

Filter by brand is working with the latest commit f589b8f givanz/landing@40ba1e3

@Twilight-Computer
Copy link
Author

I have applied this, but still not seeing filter by Brand working. When filter is selected, the result in the browser shows - https://www.twilightcomputer.com.au/shop/desktops?filter[manufacturer_id][]=6 - seems like a strange character at the end of the string.?
Result shows all entries in the Category.

@givanz
Copy link
Owner

givanz commented Apr 2, 2024

Can you please redownload latest.zip and override app/components, admin/sql and system folders if possible overwrite all files except config and public/themes and clear cache? make sure that storage/model/app folder has write permission and is emptied on clear cache.

The strange characters are [ and ] that are urlencoded by the browser and transformed to %5B and %5D.

@Twilight-Computer
Copy link
Author

I have replaced all files except config and public/themes (and public/media). Cleared cache, restarted httpd and php-fpm.
Folder storage/model/app has 777 permissions. However on clearing cache, the contents of this folder is not deleted. Is this what you meant by "make sure that storage/model/app folder has write permission and is emptied on clear cache." ?
I have manually deleted the contents, and it gets rebuilt on use of the Website unfortunately still not showing filtered results.

@Twilight-Computer
Copy link
Author

Not that I am seeing this entry multiple times in the log, but the actual file does not exist ?
PHP Deprecated: ucfirst(): Passing null to parameter #1 ($string) of type string is deprecated in /var/www/html/twilight/storage/compiled-templates/admin_1_default_product_product.html on line 2022

@givanz
Copy link
Owner

givanz commented Apr 3, 2024

the contents of this folder is not deleted

If files are not deleted it fails silently, I added some warnings in tools > cache for cache folders that are not writable.

Do you use the code from github or latest.zip? Not all latest changes are on github to allow some testing before push, please update the code from latest.zip

Also public/js/filter.js is not the latest version with ajax support as it refreshes the page on filter change.

@givanz
Copy link
Owner

givanz commented Apr 3, 2024

Not that I am seeing this entry multiple times in the log, but the actual file does not exist ?

Thanks for the bug report, It's fixed in the latest update, changed file /system/functions.php

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants