feature request - provost page changes

Bug #501221 reported by Teffania
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Canon Lore
Triaged
Wishlist
Unassigned

Bug Description

low priority: if easy to do

A version of the provosts page available that excludes mundane names and so is available without needing the password.Probably linked to from the provosts linkage page.<http://lochac.sca.org/canon/provost.php>,

Revision history for this message
Teffania (teffania) wrote :

Additional changes requested - priority medium to high - this is probably making the provist's life quite difficult.

From the provost of scribes: <provost at lochac.sca.org>
"One thing I would like to do on the Scribes part of the site is limit my search of awards to all of those who have received awards in the last six months or some given time period. I update the database as often as possible for scribes and I use Canon to do that but I either have to do a find for 2010 or go through every Australian/ NZ award to make sure my database is correct"

Teffania Canon adds:
May be best to talk further with provost about what would be good. For example, they don't do scrolls for every award, so probably could limit the list of awards to the scroll giving ones and only in lochac.

In the longer term, I'd also like the provost and other advanced users to be able to get the know the shorthand person id field optionally on some reports. I'd also like some of the expanded notes features to exist so I could share different bits of the notes with the provost, but not all of them, and not with all users of the provost's report (we might need ultimately to split that report into two or three forms - a true provost's report, a royalty report, and a general herald/seneschal/baron/award auditor form). That's a lot of stuff to consider, and probably more long term, but I'm mentioning it because it might be worth keeping in mind while doing the immediate fix of limiting awards to the last 6 months as it may affect desing descisions.

see also:
notes feature expansion request: https://bugs.launchpad.net/gratian/+bug/501516

Revision history for this message
Paul Harrison (paul-francis-harrison) wrote :

The original bug description sounds pretty simple, as does the request by the provost of scribes. Comment #1 suffers from scope creep, which I'm going to disregard.

The following will be simple:

* have the provost page work without a password, but not display mundane names
* limit results by date and/or sort by date

Changed in canonlore:
assignee: nobody → Paul Harrison (paul-francis-harrison)
Revision history for this message
Teffania (teffania) wrote : Re: [Bug 501221] Re: feature request - provost page changes

On 28 June 2010 11:29, Paul Harrison <email address hidden> wrote:
> The original bug description sounds pretty simple, as does the request
> by the provost of scribes. Comment #1 suffers from scope creep, which
> I'm going to disregard.

good - the extra comments are just fantasy wishlist items, simply put
there so that you can know if anything you do might render them
impossible or massively more difficult.

> The following will be simple:

> * limit results by date and/or sort by date

This item carries the priority, as it will make the provost's life
significantly easier.

> * have the provost page work without a password, but not display mundane names

I presume "have the provost page work without a password, but not
display mundane names" means
IF password given, display mundane names, IF password not given, don't
display mundane names.

This is a wishlist item, (unlike the provost's request), but if it's
simple, I'd like it, because it makes it possible for more people to
know about this page (becomes general knowledge), and for the
aforesaid seneschals, barons, etc to access the printable version,
even in a hurry when they can't remember the password.

Are there any security implications if we actually have a link to this
page from the main canonlore? (ie does it make it more suceptible to
people trying to hack it and gain our confidential data - the mundane
names)

Teffania

Teffania (teffania)
Changed in canonlore:
importance: Undecided → Medium
Revision history for this message
Eric TF Bat (bat-flurf) wrote :

New features:

- Hid all the complexity behind an "advanced search" link.
- Put the link to the provost page in the sidebar.
- Running with a blank password will produce a report without mundane names.
- The "don't show aliases" option now works properly
- Can specify a range of years to limit the awards to.

As a side-effect, the code now only produces results for people with one or more awards. This is a limitation I'd like to remove, but the code desperately needs refactoring so I'd rather do that before I add any more features.

Revision history for this message
Teffania (teffania) wrote :

Limit to Lochac awards for which scrolls are made.

Teffania to consult with provost to see which these are, but thinks it might be:

peerages of merit:
pelican #95, laurel #94, chivalry #96

Grants of arms:
plain GOA #86, red wyvern #125, cockatrice #124, silver pegasus #123

plain Award of arms: #52
perhaps people whose first award is armigerous but not a plain AOA.

And scrolls for counties #103, duchies #104 are being considered.

Revision history for this message
Teffania (teffania) wrote :

more GOA's: LCOA #85, white scarf #89

Revision history for this message
Teffania (teffania) wrote :

Priority dropped now that provost can limit listing to this year.
Assigned to Teffania as she is to ask provost what awards to be listed.

Changed in canonlore:
assignee: Paul Harrison (paul-francis-harrison) → Teffania (teffania)
importance: Medium → Low
Revision history for this message
Teffania (teffania) wrote :

Checked with provost of scribes, above list is correct.

Also add viscounty#102 (even though there currently aren't any beign created in Lochac).

The most complicated bit of the above is confirmed - when someone's first award is not an AOA, but is of the armigerous rank, the rank is granted with that award, and the person gets a scroll. This will be relativey rare, but will occur.

In an attempt to do this technically:
awards this applies to:
when realm = lochac
and rank =armiger <http://lochac.sca.org/herald/type_rank>
and no earlier award of rank armiger exists (recieved date is lower than all other awards of rank armiger)

I know this'll be tricky, but it sounds like it might be possible to me.

Changed in canonlore:
status: New → Incomplete
status: Incomplete → Triaged
importance: Low → Wishlist
assignee: Teffania (teffania) → nobody
Revision history for this message
Teffania (teffania) wrote :

Katherine Kerr (compiler of the royal book, which now contains the OP) has requested greater refinement on the date features - to allow last 6 months, not simply the year range.

This is probably superceeded if the other changes she has requested are implemented.
(see https://bugs.launchpad.net/canonlore/+bug/650763)

However, I think the provost of scribes is also likely to think in these terms sometimes, rather than a year range which currently in early June would require loading all awards 2010-2011, ie 17.5 months worth to get 6 months.

Revision history for this message
Paul Harrison (paul-francis-harrison) wrote :

Ok, I've added month specification to the provost report.

Note: I've suppressed display of awards with date ????-??-?? when a from-date is specified.

Revision history for this message
Teffania (teffania) wrote :

nice!

thankyou.

On 4 April 2011 18:51, Paul Harrison <email address hidden> wrote:
> ????-??-??

--
. ___
 {o,o}           The blog you are not looking for
 |)__)                   is definitely not at
 -"-"-                   http://teffania.blogspot.com

Revision history for this message
Eric TF Bat (bat-flurf) wrote :

It's a fair cop. It really is an appalling piece of shite, that script. Thrown together on a whim at the request of a provost of scribes about mumbleteen years ago, and never really fixed. It should be done via an XSLT stylesheet, perhaps, but who ever has time to do things right? Doing them at all takes 180% of available time and effort...

Revision history for this message
Canon Herald (canon-lochac) wrote :

On 4 April 2011 18:51, Paul Harrison <email address hidden> wrote:
> Ok, I've added month specification to the provost report.

This (or some other change made today) appears to have crashed the
provost's report.

Warning: strstr() [function.strstr]: Empty delimiter. in
/sca-data/disk/home/sca/public_html/canon/php/provost-common.php on
line 168

Message appears to not happen when both year and month are specified.
Happens with simple search and advanced search both with and without
password in all combinations tried without both month and year
specified. more instances of error message where years not filled in
or simple search.

Suggest actually checking result of programming in web browser at end
of uploading - basic testing. Then ask someone on canonlist to test
it for you, thus doing extra testing. Eventually one of these bugs
will be one that crashes the whole of canonlore, and you don't want
that to be the night before a camping event. You can tell how much
people love it by how many complaints we get if it is down.

Teffania

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.