Page MenuHomePhabricator

History icon url in the content browser of diffusion does not include file path
Closed, ResolvedPublic

Description

I found this issue on our local Phabricator installation, but you can also see a live example on https://secure.phabricator.com

Steps to reproduce:

  1. Visit https://secure.phabricator.com/diffusion/P/browse/master/src/ (this bug is not occurring in the root of the repo)
  2. Click on the history icon of a file or directory. It's the most left icon in the image below.
  3. There is no history, because the url does not include the path. The url is https://secure.phabricator.com/diffusion/P/history/master/__phutil_library_init__.php, but it should be https://secure.phabricator.com/diffusion/P/history/master/src/__phutil_library_init__.php

Schermafbeelding 2015-11-16 om 11.42.47.png (59×716 px, 19 KB)

Event Timeline

dennis updated the task description. (Show Details)
dennis added a project: Diffusion.
dennis added subscribers: dennis, claassen.
epriestley renamed this task from Bug: history icon url in the content browser of diffusion does not include file path to History icon url in the content browser of diffusion does not include file path.Nov 16 2015, 4:26 PM
epriestley claimed this task.
epriestley triaged this task as Normal priority.

I was able to reproduce this easily by following your instructions. We'll get this fixed. Thanks for the report!

This should be fixed at HEAD. Let us know if you run into anything else.