Most Used Tags

[Michael Collins]

OK - thank you - I will get back to you.

[AA15]

Hi @Adriano Faria! This awesome plugin does not work with PHP 8 or PHP 8.1. Is it possible you could please kindly update?

[Adriano Faria]

8 minutes ago, AA15 said:

Hi @Adriano Faria! This awesome plugin does not work with PHP 8 or PHP 8.1. Is it possible you could please kindly update?

I’ll test it tomorrow and will update it if necessary. 

[Adriano Faria]

11 hours ago, AA15 said:

Hi @Adriano Faria! This awesome plugin does not work with PHP 8 or PHP 8.1. Is it possible you could please kindly update?

Do you have any specific error message? Anything in the logs? It works fine to me on IPS 4.7.3 with PHP 8.1.11 or 8.0.24:

[AA15]

2 hours ago, Adriano Faria said:

Do you have any specific error message? Anything in the logs? It works fine to me on IPS 4.7.3 with PHP 8.1.11 or 8.0.24:

Yes, these are the errors in the system logs. The type of error is "uncaught_exception"

 

Quote

The URL of page the error occurred on was https://imagez.com/index.php?app=core&module=system&controller=widgets&do=getBlock&blockID=app_tags_tags_u1m191eob&pageApp=core&pageModule=discover&pageController=streams&pageArea=sidebar&orientation=vertical&csrfKey=6ad540ea6c346fd1d98889fd6b062c82

TypeError: count(): Argument #1 ($value) must be of type Countable|array, int given (0)
#0 /home/imagez/public_html/applications/tags/widgets/tags.php(94): count(19)
#1 /home/imagez/public_html/system/Widget/Widget.php(833): IPS\tags\widgets\_tags->render()
#2 /home/imagez/public_html/system/Widget/Widget.php(888): IPS\_Widget->_render()
#3 /home/imagez/public_html/applications/core/modules/front/system/widgets.php(150): IPS\_Widget->__toString()
#4 /home/imagez/public_html/system/Dispatcher/Controller.php(107): IPS\core\modules\front\system\_widgets->getBlock()
#5 /home/imagez/public_html/applications/core/modules/front/system/widgets.php(38): IPS\Dispatcher\_Controller->execute()
#6 /home/imagez/public_html/system/Dispatcher/Dispatcher.php(153): IPS\core\modules\front\system\_widgets->execute()
#7 /home/imagez/public_html/index.php(13): IPS\_Dispatcher->run()
#8 {main}

Backtrace

Quote

#0 /home/imagez/public_html/init.php(1029): IPS\_Log::log('TypeError: coun...', 'uncaught_except...')
#1 /home/imagez/public_html/system/Widget/Widget.php(939): IPS\IPS::exceptionHandler(Object(TypeError))
#2 /home/imagez/public_html/applications/core/modules/front/system/widgets.php(150): IPS\_Widget->__toString()
#3 /home/imagez/public_html/system/Dispatcher/Controller.php(107): IPS\core\modules\front\system\_widgets->getBlock()
#4 /home/imagez/public_html/applications/core/modules/front/system/widgets.php(38): IPS\Dispatcher\_Controller->execute()
#5 /home/imagez/public_html/system/Dispatcher/Dispatcher.php(153): IPS\core\modules\front\system\_widgets->execute()
#6 /home/imagez/public_html/index.php(13): IPS\_Dispatcher->run()
#7 {main}

 

Edited October 24, 2022 by AA15

[Adriano Faria]

Ok, thanks. New version pending approval.

[Adriano Faria]

FROM NOVEMBER 1ST TO NOVEMBER 30TH.

[Michael Collins]

On 8/18/2022 at 7:48 PM, Adriano Faria said:

your image also shows it is still in use. Provide an ACP account and URL and I'll take a look directly in your database.

Hi @Adriano Faria, I have sent you a message with these details.

[Adriano Faria]

On 8/18/2022 at 2:10 PM, Michael Collins said:

However, when I delete all instances of a single tag across a number of posts - I see that the tag name still appears with a count in the most used tags page

How are you deleting these tags? Directly in the database table?

[Michael Collins]

I went into the posts in question, edited them and deleted the unrequired tags.

[Adriano Faria]

As I said before, this resource uses the core_tags table to select the data:

		$rows = array();
		foreach( \IPS\Db::i()->select( 'tag_text, count(tag_text) as cnt, SUM(tag_prefix) as prefixed, MAX(tag_added) as last',  'core_tags', array( 'tag_meta_app=?', \IPS\Request::i()->fromApp ), $order, null, 'tag_text' ) as $row )
		{
			$rows[] = $row;
		}

I don't see how this could be showing records that doesn't exists.

[Michael Collins]

Ok, so you mean the data may be still in the table even though the tags are removed? what do you suggest? Did you manage to have a look at the database like you mentioned previously?

[Adriano Faria]

2 minutes ago, Michael Collins said:

Did you manage to have a look at the database like you mentioned previously?

Nope, will do. Making a test in my online board first.

[Adriano Faria]

On 8/18/2022 at 3:42 PM, Michael Collins said:

Attached are a screenshot of two of the tags that are inaccurate - and what happens when I click on the top tag:

Thanks - Mike.

On a quick query, I see both:

Make sure these topics are in a forum you can view. Theses topics can't be archived as they wouldn't appear in search.

[Michael Collins]

None of them are archived. We used the most used tags app to find tags we didn't want and removed them from all the posts it said they were in, that's all the editing we did.

[Adriano Faria]

Well, then you have an issue to IPS handle. My app is showing records that exists, as you can see above.

No error on my app. 👍

Request support and say that are tags in topics that aren't showing in search (tags) results.

[Michael Collins]

Ok, thanks for looking Adriano, I will contact support.

[Michael Collins]

@Adriano Faria Is there any plan for pagination of the most used tags results? Or some sort of filtering or a-z jumping?

[Adriano Faria]

9 minutes ago, Michael Collins said:

Is there any plan for pagination of the most used tags results? Or some sort of filtering or a-z jumping?

Do you mean to add a pagination instead of only show X tags here?

[Michael Collins]

Yes, exactly, we have thousands of tags so scrolling through them isn't very user friendly.

[Adriano Faria]

Just now, Michael Collins said:

Yes, exactly, we have thousands of tags so scrolling through them isn't very user friendly.

That would require basically a redoing of this due to the query it uses.

I’ll give it a try in the forthcoming days.