Based on #1772300: RSS feed for Git commits links to wrong URL, comma in repo_id can make i.e. urls do not work.

Comments

drumm’s picture

drumm
he/him
Location NY, US
commented
Status: Active » Postponed (maintainer needs more info)

I'm not seeing any use of repo_id in Views in drupalorg modules.

perignon’s picture

perignon commented
Priority: Normal » Major

Trying to find where to get action on this bug on D.O

Links contained in the RSS feed of Git commits are broken because of a "%2C" being added in the URL.

perignon’s picture

perignon commented
Status: Postponed (maintainer needs more info) » Active
Related issues: +#2373389: RSS Feed of User Commits have invalid links

Adding related issue filed against D.O infrastructure

drumm’s picture

drumm
he/him
Location NY, US
commented

Example:

https://www.drupal.org/commitlog/commit/71%2C491/ed7a04a7e89fa7f1c9df313... vs. https://www.drupal.org/commitlog/commit/71491/ed7a04a7e89fa7f1c9df313f04...

The %2C is a comma, and should be removed.

drumm’s picture

drumm
he/him
Location NY, US
commented
Project: Drupal.org customizations » Version Control / Project* integration
Version: 7.x-3.x-dev » 7.x-1.x-dev

And the example feed: https://www.drupal.org/user/1547200/track/code/feed

This View is in code at views/default_views/vc_project_user_commits.view.php. The other default Views there should be checked too.

perignon’s picture

perignon commented

Thank you for your prompt attention!

marvil07’s picture

marvil07 commented
Project: Version Control / Project* integration » Drupal.org customizations
Version: 7.x-1.x-dev » 7.x-3.x-dev
Status: Active » Closed (works as designed)

I guess this was meant to be on #2270699: Do not use separator on repo_id view field., and in drupalorg there is nothing to do.
Please reopen if something in drupalorg is needed.

perignon’s picture

perignon commented

So where/how do we get this fixed?

perignon’s picture

perignon commented

NM... I found the other thread.