avatar brianteeman
brianteeman
18 Oct 2016

I have seen this a few times today where you cant submit a test result because of a missing sha

Example Pr is https://issues.joomla.org/tracker/joomla-cms/12431

You can see instead of the SHA a link which gives a 500

avatar brianteeman brianteeman - open - 18 Oct 2016
avatar mbabker
mbabker - comment - 18 Oct 2016

Intentionally added - 9ef60aa - #881

It's only a partial fix but it's addressing the issue of an invalid data record.

I think I might have found the path though that can produce this issue. We've got two JavaScript methods submitting the test result; one includes the SHA and the other doesn't (this one is JTracker.submitTestWithComment), so I'm trying to trace my way back up that to get the missing data.

avatar mbabker
mbabker - comment - 18 Oct 2016

If I've got the right code path figured out here, this is how you can test whether it's fixed or not.

First, ensure the browser cache is refreshed as the jtracker.min.js file is changed.

Set the test result to (un-)successful, add a comment using the form at the bottom of the page, and submit with that button. It should add the comment and the test result.

With the test result option set to "Not tested", submitting the comment by that button at the bottom of the page should NOT trigger the script to submit a test result.

avatar brianteeman
brianteeman - comment - 18 Oct 2016

Thanks. Will test in the morning. Can't test from my phone.

avatar brianteeman
brianteeman - comment - 18 Oct 2016

seems ok if I tested correctly on 12431


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/jtracker/895.

avatar mbabker
mbabker - comment - 18 Oct 2016

Looks like all is fine.

avatar mbabker mbabker - change - 18 Oct 2016
Status New Closed
Closed_Date 0000-00-00 00:00:00 2016-10-18 13:04:16
Closed_By mbabker
avatar mbabker mbabker - close - 18 Oct 2016

Add a Comment

Login with GitHub to post a comment