Fixed Missing relation for trending content handler

[Kirby]

UBS Version

2.3.8

XenForo Version

2.3.6

PHP Version

8.3.20

Database & Version

MariaDB 10.11.11

Are there any errors being throw?

No

XenAddons\UBS\TrendingContent\BlogEntryHandler::getEntityWith() does not include relation Blog, this causes additional queries for each distinct blog as BlogEntry canView calls Blog canView

 

[Bob]

Whoops! It does now Nice catch!

    public function getEntityWith(string $style): array
    {
        $visitor = \XF::visitor();

        return [
            'Blog',
            'Category',
            'Category.Permissions|' . $visitor->permission_combination_id
        ];
    }

Also added the missing Blog relation to the Featured Content Handler XenAddons\UBS\FeaturedContent\BlogEntryItem::getEntityWith()

    public function getEntityWith(): array
    {
        $visitor = \XF::visitor();
    
        return [
            'Blog',
            'Category',
            'Category.Permissions|' . $visitor->permission_combination_id,
            'User',
        ];
    }

 

[Kirby]

CoverImage is also missing for (at least) UBS and SC.

 

[Bob]

CoverImage is also missing for (at least) UBS and SC.

Got them added to both Trending Content Handler and Featured Content Handler!

Note to self: Update CAS, EMS, IMS, LD, RMS as well...