Printable View
[#GDT-419942]: Error in Tags Plugin - Wed 30/01/2019 9:22 PM
/\ I have sent this in almost 2 weeks ago, also sent a follow up on - Fri 8/02/2019 11:00 PM and have still not heard anything back from anyone, are we able to check if it is in the system or should I start again with a new ticket ?
regards
Pete
Pete, may I know the problem and result.
By the way, what does /\ mean?
/\ Points aboveQuote:
Originally Posted by Pete Young
The error was that the meta title did not resolve properly and it did not show the title of the tag, resulting in poor / zero results in search as none of the tags would have indexed correctly, I have deleted the video of the error that I made but I can whip up another to show you where there error was if you need it.
Pete
Yes, i found some problems in tag cloud plugin include title problem you mentioned.
If you go to Basic settings>tag cloud , you'll see two important parameters.
1. Tag minimum length. default value is 3
It means the minimum tag length for using by the plugin is 3 characters.
Tag example : Sell car in sydney
The plugin gathers all listings which contain these words: sell,car,sydney
It should not gathers listings with 'in' word because minimum tag length is 3
But unfortunately plugin does not work properly with this parameter and you can imagine what will happen.
Solution: Go to plugins/tag_cloud/rlTagCloud.class.php
Find: tc_tag_min_symbols
Replace with: tc_tag_min_length
2. Banned words
If you add words like 'the,this,that,what,which,who,...' and your tag cloud is 'the best car price'
So it prevents gathering listings which contain 'the' word.
It seems, this part of code work properly, but do not forget to use this parameter.
3. Missing title as you mentioned. Go to plugins/tag_cloud/tags.inc.php
Find:
Replace with:Code:Only registered members can view the code.
I think tag cloud needs a whole investigation.Code:Only registered members can view the code.
appreciate the tips, thank you
also added this link here for others as a sidenote to tags
https://forum.flynax.com/showthread....-order-in-tags
for your number 3 option I have this added to the bottom as the fix ?
[QUOTE]// by XXX
$bread_crumbs[0] = array(
'name' => $tag_info['Tag']
);
$page_info['meta_title'] = $bread_crumbs[0]['name'] . '
yours seems a simpler fix ?
[QUOTE=Pete Young;25052]for your number 3 option I have this added to the bottom as the fix ?
Well done! yours is good , but i usually try with minimum changes to the main code.Quote:
// by XXX
$bread_crumbs[0] = array(
'name' => $tag_info['Tag']
);
$page_info['meta_title'] = $bread_crumbs[0]['name'] . '
yours seems a simpler fix ?
[QUOTE=Bahram Soltanirad;25053]That is not my code so do not take credit that as it was a support fixQuote:
Originally Posted by Pete Young
[QUOTE=Pete Young;25063]Ok, then thanks for sharing it.Quote:
Originally Posted by Bahram Soltanirad
This coding has changed in 4.7.2 and now no longer works properly and the default flynax code is still broken, could flynax please give a correct edit to the code to show the title please ?
Hello,
Could you detail on how the meta title should look on tag page ?
Hello Rudi Please see Image below (tag name is missing from page title), Bahrams fix did fix this in an earlier version but the coding has since changed and fix no longer works? thanks peteQuote:
Originally Posted by Rudi
https://forum.flynax.com/attachment....4&d=1602032777
Ok,
Here is the solution:
go to plugins > tag_cloud > tags.inc.php
below:
add:PHP Code:Only registered members can view the code.
PHP Code:Only registered members can view the code.
[QUOTE=Rudi;31065]Ok,
Here is the solution:
go to plugins > tag_cloud > tags.inc.php
below:
add:PHP Code:Only registered members can view the code.
/QUOTE]PHP Code:Only registered members can view the code.
Thanks Rudi, that works even better than I hoped for as it also covers the H1, Thank You for your help.
Edit. I did find one error, if there is only only one item found it repeats the title rather than count ?
copywriting for copywriting < 1 i item found
4 Listing(s) Found. for dating < more than 1 item found
Ok, use the code below without the second line (Corrected)
Ok thanks Rudi Will try that out.
That sort of fixed the H1 / web page but what I did not notice is in the title it also repeats as well if there is a single listing found.
With the last line removed
(with one listing found)
Page Title > copywriting copywriting <<< Shows double in title, so still an error here ?
H1 > copywriting
More than one listing found
Page Title > Search For relationships
H1 > 8 Listing(s) Found.
With the last line included
(with one listing found)
Page Title > copywriting copywriting <<< Shows double in title, so still an error here / Same as no bottom line so an error regardless if one listing?
H1 > copywriting for copywriting <<< as before
More than one listing found
Page Title > Search For relationships << stays the same OK
H1 > 8 Listing(s) Found. for relationships << the best outcome and very good.
---
With the bottom line works better all round but problems if one listing found both with or without the bottom line (with one listing) as title is still wrong.
Would like to keep the bottom line but fix the single problem ? if possible.
I've corrected it on your end
replaced:
with:PHP Code:Only registered members can view the code.
PHP Code:Only registered members can view the code.
Spot on thank you Rudi, that is perfect, appreciate your time.
Pete