Ticket #869 (closed task: wontfix)

Opened 4 years ago

Last modified 3 years ago

Browse by Date: use most recent pub date as the endDate for "past week", "past month", and "past 3 months"

Reported by: russ Assigned to:
Priority: high Milestone:
Component: ambra Version: 0.8.2.1
Keywords: Cc:
Blocking: Blocked By:

Description

find most recent pub date, and use that as the end date

some journals don't publish weekly.

Dependency Graph

Change History

06/19/08 15:07:22 changed by amit

  • owner changed from alex to russ.
  • type changed from enhancement to clarification.
  • blocking changed.
  • blockedby changed.
  • milestone set to 0.9.0.

Not sure I have understood this correctly. We have the concept of Past Month and Past 3 Months. Journals can choose to make one of them the default. Why do we need to do it on a weekly basis?

I think we need to get away from Browse by data altogether. I would be curious to see what the user stats here are. With the ability to do search on date ranges why is this needed?

06/19/08 15:19:58 changed by ronald

See source:private/russ/benchmark.xls, the first sheet - seems to get used.

06/20/08 11:24:43 changed by russ

  • owner changed from russ to amit.
  • type changed from clarification to task.
  • summary changed from Browse by Pub Date: make default week the most recently published week to Browse by Date: use most recent pub date as the endDate for "past week", "past month", and "past 3 months".

addition to past month and past 3 months we also have past week. this is nothing new.

"journals pubbed in the last week" is the default response to browse.action?field=date if no day, month, or year parameters exist.

it's what we point the "browse by date" link to in all the journals.

if nothing has been published in the last week, it's blank, which is lame.

we'd the default case to find the most recent pub date, and use that as endDate for the week calculation.

in fact, it would probably be good use to the most recent pub date as the endDate for all 3 "past x" actions.

06/20/08 11:36:17 changed by amit

  • owner deleted.
  • priority changed from unassigned to high.
  • milestone deleted.

So we need to add 'last published date' somewhere (probably to Journal).

06/20/08 16:47:07 changed by ronald

First, I would argue that "past week" means exactly that, not some random previous week. If journals don't publish weekly, then just have browse-by-date point to previous month or something. Also, it's not hard to add other, fixed, previous-interval options.

From the implementation side a previous-x-starting from last-published would be far more work than just adding another date field, because currently the queries currently always know what date to search for, which will not be the case anymore. This could have a noticeable hit on performance and will require a whole bunch of rework on the caching side.

I.e. this would not be a "simple" change.

06/20/08 16:55:46 changed by amit

:) Seem my first comment.

12/20/08 11:14:06 changed by amit

  • status changed from new to closed.
  • resolution set to wontfix.

On further analysis, this is not a good idea for general implementation. It overloads the idea of 'recently published' with concept of the 'latest week' which is going to be very confusing. Browse by date should remain as is. It would be better to add 'Browse recently published articles' as an option rather than changing the idea of 'current date'. If that is in line with the thinking, please open an enhancement ticket on the same.