Skip to content

Comments

[5.0] Remove annoying "user not found" message#41048

Merged
HLeithner merged 1 commit intojoomla:5.0-devfrom
Fedik:user-not-found-msg
Jun 26, 2023
Merged

[5.0] Remove annoying "user not found" message#41048
HLeithner merged 1 commit intojoomla:5.0-devfrom
Fedik:user-not-found-msg

Conversation

@Fedik
Copy link
Member

@Fedik Fedik commented Jun 26, 2023

Pull Request for Issue # .

Summary of Changes

Remove annoying "user not found" message. That show up every time when open list of content that created by removed user.

Testing Instructions

Apply patch.
Create testing user.
Create an article with this user.
Delete testing user.
Open just created article to edit.

Actual result BEFORE applying this Pull Request

Message, kind of "cannot load X user"

Expected result AFTER applying this Pull Request

No message

Link to documentations

Please select:

  • No documentation changes for docs.joomla.org needed
  • No documentation changes for manual.joomla.org needed

@richard67
Copy link
Member

Would this solve issue #41008 ?

@laoneo
Copy link
Member

laoneo commented Jun 26, 2023

And it needs some documentation.

@coolcat-creations
Copy link
Contributor

It looks more like a quick fix then #40751 - I feel like the other PR does not only solve the symptoms but also gives the opportunity for other extensions to hook in?

@Fedik
Copy link
Member Author

Fedik commented Jun 26, 2023

Would this solve issue 41008 ?

Sounds like

And it needs some documentation.

I not sure, I do not really see a need in it.
Most of the time it was not much helpfull message.

@HLeithner HLeithner merged commit 858c25b into joomla:5.0-dev Jun 26, 2023
@HLeithner
Copy link
Member

thanks, can we please get a documentation entry. and hope fully a full fix for "auto" replacement soon.

@Fedik Fedik deleted the user-not-found-msg branch June 26, 2023 08:25
@Fedik
Copy link
Member Author

Fedik commented Jun 26, 2023

Okay, added

@VillaesterModerneMedien

Thank you for removing the message, but the problem with the IDs is not solved for me....

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants