Ticket #911 (closed defect: fixed)

Opened 9 months ago

Last modified 4 months ago

"First" concept for representations is bogus. Replace with something else.

Reported by: pradeep Assigned to: ronald
Priority: high Milestone: 0.9.1
Component: ambra Version: 0.9-SNAPSHOT
Keywords: Cc:
Blocking: Blocked By:

Description 

Life is hard, and then you die wrote:
> On Tue, Apr 15, 2008 at 10:31:16PM -0700, Pradeep Krishnan wrote:
>> Currently the 'fetchFirstRepresentation' on an article is really a 
>> fetch-Any-Random-Representation. Since we are doing a bit of 
>> data-migration around article representations, I was wondering if we 
>> need to create an ordering of representations. What does "first" signify 
>> in this case any way? Is that expected to be the first in the article Zip?
> 
> I don't think "first" makes any sense, and neither does ordering of
> representations. Things like fetch-smallest or fetch-original can make
> sense, though the semantics are very app specific.
> 
> Hmm, it's even on the article-list page: "View first representation".
> I'd vote to get rid of that completely, or more likely replace with
> "View HTML representation".
>

Dependency Graph

Change History

04/15/08 23:56:07 changed by pradeep

  • owner deleted.
  • type changed from unassigned to defect.

08/18/08 08:33:02 changed by amit

  • milestone set to 0.9.1.

09/08/08 16:00:35 changed by amit

  • owner set to ronald.

09/08/08 16:00:44 changed by amit

  • priority changed from unassigned to high.

09/08/08 17:26:19 changed by ronald

  • status changed from new to assigned.

09/20/08 05:05:36 changed by ronald

  • status changed from assigned to closed.
  • resolution set to fixed.

(In [6438]) Rename fetchFirstRepresention and fetchFirstObject to fetchSingleRepresention to be more descriptive of what this is actually used for and because the concept of "first" makes no sense among representations (which are a Set).

This closes #911, as the other use of this function that had implicit assumptions about what the first representation was supposed to mean has disappeared with the removal of articleList.ftl in r6318.