Skip to content

fix: Get Sanitized Name #2030

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Jun 18, 2025
Merged

fix: Get Sanitized Name #2030

merged 1 commit into from
Jun 18, 2025

Conversation

solracsf
Copy link
Member

@solracsf solracsf commented Jun 17, 2025 •
edited
Loading

Should fix the error.

{
  ...
  "app": "circles",
  "method": "GET",
  "url": "/ocs/v2.php/apps/dashboard/api/v2/widget-items?widgets%5B%5D=circles",
  "message": "Parameter \"guestName\" for route \"core.guestavatar.getavatar\" must match \"[^/]++\" (\"Projet de Fusion ARC CSEM- OS/POs\" given) to generate a corresponding URL.",
  ...
}

MariaDB [nextcloud]> SELECT display_name, sanitized_name  FROM oc_circles_circle  WHERE display_name = 'Projet de Fusion ARC CSEM- OS/POs';
+-----------------------------------+-----------------------------------+
| display_name                      | sanitized_name                    |
+-----------------------------------+-----------------------------------+
| Projet de Fusion ARC CSEM- OS/POs | Projet de Fusion ARC CSEM- OS-POs |
+-----------------------------------+-----------------------------------+
1 row in set (0.000 sec)

@solracsf
fix: Get Sanitized Name
6d992a9 
Signed-off-by: Git'Fellow <[email protected]>
@solracsf
Copy link
Member Author

/backport to stable31

@solracsf
Copy link
Member Author

/backport to stable30

@solracsf solracsf added this to the Nextcloud 32 milestone Jun 17, 2025
@ArtificialOwl ArtificialOwl merged commit 6b39516 into master Jun 18, 2025
24 of 42 checks passed
@ArtificialOwl ArtificialOwl deleted the getSanitizedNameFix branch June 18, 2025 11:12
This was referenced Jun 18, 2025
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ArtificialOwl ArtificialOwl ArtificialOwl approved these changes

@provokateurin provokateurin provokateurin approved these changes

@come-nc come-nc come-nc approved these changes

Assignees
No one assigned
Labels
3. to review bug
Projects
None yet
Milestone
Nextcloud 32
Development

Successfully merging this pull request may close these issues.
4 participants
@solracsf @ArtificialOwl @provokateurin @come-nc