Make "edit" work - or at least allow cancellation



5 years ago
11 months ago


(Reporter: dholbert, Unassigned)



Firefox Tracking Flags

(Not tracked)




5 years ago
I'm told that if you attempt to use the "Edit" link on an already-filed PTO report in our PTO webapp[2], it *actually* creates a duplicate PTO filing, and both reports will be counted against your PTO balance.

(You might try to do this e.g. to upgrade a half-day to a full day off, or something similar.)

This is apparently hard to fix, because the app is fragile; instead of "edit", we're supposed to email payroll@.

However, for people who don't know about this bug, this is a pretty massive footgun that can result in completely unnoticed loss of PTO.[2]

SO: assuming we can't fix the double-entry, can we please *disable* the "Edit" button, since AFAICT there is no valid use for it currently other than to (unintentionally) trick people into losing PTO?

[2] e.g. in auditing my PTO reports and my paystubs, I just now discovered that I likely lost 4 hours of PTO due to an "edit"-triggered double-report last July. (Doris is looking into it, which is awesome, so it presumably will be corrected in this one instance -- but only because I investigated & reported the problem.)

Comment 1

5 years ago
(Sorry, the first "[2]" should've been "[1]", obviously)

Comment 2

5 years ago
Looks like this needs some help to get the right people CC'd. Peter, do you know who might be able to disable this button?

(I know it's fragile, but surely there has to be an easy way to just hide this feature and stop people from flushing PTO hours down the toilet without realizing it.  e.g. modifying "edit.php" to hide the page contents, and perhaps suggest emailing payroll@?)
Flags: needinfo?(peterbe)
I've actually never worked or even seen the files for the PTO app. I did work on the replacement PTO app but it never got deployed. 

I think Lonnen is now an owner of the old app.
Flags: needinfo?(peterbe) → needinfo?(chris.lonnen)

Comment 4

5 years ago
not my circus, not my monkey
Flags: needinfo?(chris.lonnen)

Comment 5

5 years ago
Do you know who else might have access/authority to tweak this tool, though?  (or, do you know someone who you think would know the right person to poke?)

(Also, good news: there's a chance that the underlying problem here has been fixed.  At least, :djf was apparently able to edit some PTO that he took in July 2014, without the hours being counted twice.   Doris from Payroll had told me in June 2014 that the tool was still double-recording things at that point, though [and I confirmed that some PTO that I'd added in the previous year -- July 2013 -- *was* double-recorded.  That's the only instance of this that I personally have seen in action, so it's conceivable that this has been fixed since then.)

(If the underlying bug is actually fixed, then we don't need to hide the "edit" feature after all, and this bug can be WONTFIX'ed. But I don't know for sure if the underlying bug is actually fixed or not.)
Flags: needinfo?(chris.lonnen)

Comment 6

5 years ago
(It's also conceivable that Payroll might've just noticed & manually fixed-up djf's edit, I suppose, which is what saved him from the double-counting bug.  Not sure.) (/me stops waving hands)

Comment 7

5 years ago
Hi all, 
In order to record time off taken and run PTO report, payroll has no other part into the Tool. It would be nice employees can edit PTO, but we were told it is not possible. This is why we inform people if they need to make changes to already submitted PTO, to let payroll know so we can make changes manually. Otherwise, it will be double entry. 

Comment 8

5 years ago
I do not know, though I suspect the source code is in svn.

You could look at the history there and track down the last person to modify it.
Flags: needinfo?(chris.lonnen)

Comment 9

5 years ago
Thanks, lonnen & Doris.

SVN says the only person to have touched the app since 2011 has been rtucker.

rtucker, would you be comfortable modifying the PTO app to disable editing of already-submitted PTO? (Even if it just ends up meaning "comment out all the source on edit.php and replace it with instructions to email payroll".)

We should only do this if the double-reporting bug still exists, of course (which is unclear, but it sounds like it, per comment 7).
Flags: needinfo?(rtucker)
If rtucker isn't comfortable I'll do it. (Or r? his patch.)
Duplicate of this bug: 775673
My suggestion here is to make a Cancel button that sends email to payroll.
Summary: Please disable the "Edit" footgun on PTO reports → Make "edit" work - or at least allow cancellation

Comment 13

5 years ago
That would work for payroll.
Since this is a dupe of bug 775673, I'd assume this one can be R/F as well?
Flags: needinfo?(rtucker)

Comment 15

5 years ago
(In reply to Rob Tucker [:rtucker] from comment #14)
> Since this is a dupe of bug 775673, I'd assume this one can be R/F as well?

Why? As far as I understand, neither bug is fixed & neither should be marked as fixed.


5 years ago
Flags: needinfo?(rtucker)
I was under the impression that this had been resolved. I spoke with :wenzel who also had the same thought.

laura, do you know if this has been handled?
Flags: needinfo?(rtucker) → needinfo?(laura)
It's been 4 years. I'm removing the NI for laura.
Flags: needinfo?(laura)
You need to log in before you can comment on or make changes to this bug.