Fix: Atom feed broken when supplying ref_type

Vasilii Iakliushinrequested to merge
562693-fix-commit-feed-for-atom into master

What does this MR do and why?

Contributes to #562693 (closed)

Problem

The atom feed for commits fails to render entries when the ref_type parameter is provided. This occurs because the qualified ref name has a suffix .atom that is not removed.

Solution

Recalculate @fully_qualified_ref after .atom suffix was removed.

References

Commit feed broken in .atom format when passing... (#562693 - closed)

Screenshots or screen recordings

Before After
Screenshot_2025-08-25_at_13.26.20 Screenshot_2025-08-25_at_13.25.57

How to set up and validate locally

  1. Visit http://gdk.test:3000/gitlab-org/gitlab-test/-/commits/master.atom
  2. You should see entries in the feed
  3. Visit http://gdk.test:3000/gitlab-org/gitlab-test/-/commits/master.atom?ref_type=heads
  4. You should still see entries in the feed
  5. Visit http://gdk.test:3000/gitlab-org/gitlab-test/-/commits/master.atom?ref_type=tags
  6. You should see 404 page

MR acceptance checklist

Evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #562693 (closed)

Merge request reports