Skip to content

[Misc] Various small fixes#2809

Merged
mnocon merged 4 commits into
4.6from
small-fixes
Jul 3, 2025
Merged

[Misc] Various small fixes#2809
mnocon merged 4 commits into
4.6from
small-fixes

Conversation

@mnocon

@mnocon mnocon commented Jul 2, 2025

Copy link
Copy Markdown
Contributor

Target: 4.6, 5.0

A small batch of fixes I found when browsing the doc.


This isn't required in template functions or controller code, as the HTTP layer takes care of identifying the user, and automatically sets it in the repository.

If you want to identify a user with their credentials instead, provide them in the following way:

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This API has been removed in 3.3

@mnocon mnocon marked this pull request as ready for review July 2, 2025 14:17
@mnocon mnocon requested a review from a team July 2, 2025 14:17
@ezrobot ezrobot requested review from adriendupuis, dabrt and julitafalcondusza and removed request for a team July 2, 2025 14:18

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could it be 4.6.21? (You can event edit the code w/ browser console if you don't want to update your current install :p)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've decided to cheat a bit more: 3b795c8

I see no value in having the exact version there, as long it says 4.6 (and it means less maintanence for us)

Comment thread docs/api/event_reference/event_reference.md
@mnocon mnocon requested a review from adriendupuis July 2, 2025 15:14
@github-actions

github-actions Bot commented Jul 2, 2025

Copy link
Copy Markdown

code_samples/ change report

Before (on target branch)After (in current PR)

code_samples/api/public_php_api/src/EventSubscriber/MyEventSubcriber.php


code_samples/api/public_php_api/src/EventSubscriber/MyEventSubcriber.php

docs/api/event_reference/event_reference.md@15:``` php
docs/api/event_reference/event_reference.md@16:[[= include_file('code_samples/api/public_php_api/src/EventSubscriber/MyEventSubcriber.php') =]]
docs/api/event_reference/event_reference.md@17:```

001⫶<?php declare(strict_types=1);
002⫶
003⫶namespace App\EventSubscriber;
004⫶
005⫶use Ibexa\Contracts\Core\Repository\Events\Content\CopyContentEvent;
006⫶use Symfony\Component\EventDispatcher\EventSubscriberInterface;
007⫶
008⫶class MyEventSubcriber implements EventSubscriberInterface
009⫶{
010⫶ public static function getSubscribedEvents()
011⫶ {
012⫶ return [
013⫶ CopyContentEvent::class => ['onCopyContent', 0],
014⫶ ];
015⫶ }
016⫶
017⫶ public function onCopyContent(CopyContentEvent $event): void
018⫶ {
019⫶ // your implementation
020⫶ }
021⫶}

Download colorized diff

@mnocon mnocon merged commit 3292fd2 into 4.6 Jul 3, 2025
7 of 8 checks passed
@mnocon mnocon deleted the small-fixes branch July 3, 2025 09:16
mnocon added a commit that referenced this pull request Jul 3, 2025
* [Misc] Various small fixes

* PHP & JS CS Fixes

* Adjusted screenshot
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants