(bug 27111) Make cascading foreign repo's fetch description properly
authorBrian Wolff <bawolff+wn@gmail.com>
Sun, 19 Aug 2012 16:21:47 +0000 (13:21 -0300)
committerBrian Wolff <bawolff+wn@gmail.com>
Thu, 23 Aug 2012 18:44:23 +0000 (15:44 -0300)
commitca13fde652e1a1a05c835219af7b014509a1c88f
tree78e18e0db62a225485e27e71d5b42f7f51e1d4eb
parent08e7fab36770a0bd54318f95e6627baaa2de519d
(bug 27111) Make cascading foreign repo's fetch description properly

This changes action=render on file pages to include the shared image
description (if it exists). The reasons for this are two fold:

*Logically, most users would consider the shared description part of
  the content of the page, and not part of the site chrome, so it
  should be on action=render
*Foreign file repos use action=render to get image descriptions.
  if you use say en.wikipedia.org as your file repo, commons images
  get cascasively included, but all their descriptions end up
  being the noarticletext message, which is bad. This makes
  the shared image descriptions work.

Some notes on the implementation:
* on action=render the div containing the shared description
  doesn't have the id that the one on normal page view has.
  This is to prevent duplicate ids in the cascading repo scenario.
  I added a class so people can identify them if they want, and this
  allows people to easily identify the "outermost" shared description
  as it would be the only one with the id.
* This doesn't include the "shareddescriptionfollows" message
  (which is disabled by default) that would normally separate the
  local from the non-local description. I wasn't sure, but thought
  that would be considered part of the site chrome.

Patchset 2/3: trailing space/spelling mistake
Patchset 4: rebase

Change-Id: I18bdf29de62526d699740607b5015da4b01fd43d
RELEASE-NOTES-1.20
includes/ImagePage.php