I stopped tagging with the move to REL branches.

It looks like there's a bunch of work since 2.6.6; is that not ready for release?

Whoops, thanks for the reminder!
Just pushed the tag.

@Octfx I think this still needs a new release tagged. The latest in the Gerrit repo is still 2.6.5.

Change 925651 merged by jenkins-bot:

[mediawiki/extensions/WikiSEO@master] Fix Composer name and remove version number

https://gerrit.wikimedia.org/r/925651

I think you might have to make a new tag to force it to update, maybe one higher than 2.6.6 because that one was already tagged in the GitHub repo but wasn't tagged in the Gerrit one.

Never thought of this
I've updated the source on packagist, many thanks!

Ah, that makes sense. Good idea! But there's definitely some varying opinions around whether we should even use Composer to install extensions, so it might be hard to convince people to register that package name.

I was trying to get wikiseo into the mediawiki packagist namespace, but to no avail.

Change 925651 had a related patch set uploaded (by Samwilson; author: Samwilson):

[mediawiki/extensions/WikiSEO@master] Fix Composer name and remove version number

https://gerrit.wikimedia.org/r/925651

Such a configuration exists on MediaWiki 1.39, added to solve T217307

This is to remove the infobox

I think I figured out what to do is to configure $wgExtractsRemoveClasses. I wrote this config, but nothing happens.

I figured out what was causing it. On pages where templates are used (cards) that's where the description is not shown. Can I tell it to ignore templates?

I on Description2 and check page with ?action=info. And:

In T330497#8644346, @Octfx wrote:

Thanks for the report. When WikiSeoEnableAutoDescription is active, after a page has been edited, WikiSEO tries to generate a description by calling TextExtracts.
With Extension:Description2 deactivated can you check any of your pages with ?action=info and report if a description prop is listed there?

I myself use WikiSeoEnableAutoDescription = true on MediaWiki 1.39 and can confirm that it is working.

Thanks for the report. When WikiSeoEnableAutoDescription is active, after a page has been edited, WikiSEO tries to generate a description by calling TextExtracts.
With Extension:Description2 deactivated can you check any of your pages with ?action=info and report if a description prop is listed there?

I think this was fixed a while back. At least when running php extensions/WikiSeo/maintenance/GenerateDescription.php --force 0 on MW 1.39 no such exception occurs.

resolving per last comment

Current implementation support:

• dc.identifier.wikidata
• author
• description
• title
• site_name

This is more or less expected, as neither OpenGraph nor Schema.org officially support svg images.

Sorry, I've forgot to subscribe to the project! :(

This is great news. Tested it with REL1_35 and it works :) Thanks a lot!

Ah, totally forgot! Thanks @Aklapper

@Octfx: Does that mean the task status should be set to resolved or is there more to do?

This is now implemented in master, REL1_35 and REL1_39 😊

In T300587#8244408, @Prod wrote:

I don't disagree with this.

I don't disagree with this.

In T300587#8244314, @adrelanos wrote:

In T300587#8236028, @Prod wrote:

T301588 seems to address this issue on REL1_38 and newer versions of Extension:PageImages

Then all images wiki wide would need to be added class=notpageimage. That would be a crazy amount of work. I don't think that this is a good solution.

This ticket is a feature request for a global configuration option.

What's your use case for PageImages apart from populating og:image?

In T300587#8236028, @Prod wrote:

T301588 seems to address this issue on REL1_38 and newer versions of Extension:PageImages

T301588 seems to address this issue on REL1_38 and newer versions of Extension:PageImages

Change 815732 had a related patch set uploaded (by Iijima Yun; author: Iijima Yun):

[mediawiki/extensions/WikiSEO@master] DeferredDescriptionUpdate.php --force: fix type errors related to `array_shift()` call

https://gerrit.wikimedia.org/r/815732

Change 815732 had a related patch set uploaded (by Iijima Yun; author: Iijima Yun):

[mediawiki/extensions/WikiSEO@master] T313419: DeferredDescriptionUpdate.php --force: fix type errors related to `array_shift()` call

https://gerrit.wikimedia.org/r/815732

I feel like array_shift() is out of place here: even if the assignment is replaced by a simple call, dropping the first element completely, it obviously makes the supposed later test

switch ( true ) {
                        case count( $propertyDescription ) > 1:
                                // There are multiple page props with the name 'description' present
                                // This shouldn't happen, but we'll try to clean it here
                                $db->delete(
                                        'page_props',
                                        [
                                                'pp_page' => $this->title->getArticleID(),
                                                'pp_propname' => 'description',
                                        ]
                                );
                        // Intentional fall-through, as deleting all 'description' props requires inserting a new row
                        case empty( $propertyDescription ):
                                $shouldInsert = true;
                                break;

Would it help if a patch was provided? Is this something that could potentially get accepted?

Change 779112 merged by jenkins-bot:

[labs/tools/wikibugs2@master] wikibugs: relay WikiSEO to RhinosF1's channel

https://gerrit.wikimedia.org/r/779112

Change 779112 had a related patch set uploaded (by RhinosF1; author: RhinosF1):

[labs/tools/wikibugs2@master] wikibugs: relay WikiSEO to RhinosF1's channel

https://gerrit.wikimedia.org/r/779112

In T295065#7834259, @Octfx wrote:

I hadn't found a way to create img and a elements without using Html::rawElement.
But I think as they don't contain user submitted content they should be safe.