Results Window does not load session history

Project:Prolint Issue Tracker
Component:Outputhandlers
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

Something broke in release 71 or 72 in the results window. When you click on the button to load an historical session from the database, nothing appears. I select an existing session with 170 warnings recorded (I can see them on the session compare), and click Load. There is a slight pause, it appears it is reading through the warnings, but then the Prolint node appears to the left and no child-nodes nor entries in the browse. I'm guessing the subscribed signature has changed?


Comments

Comment viewing options

Select your preferred way to display the comments and click "Save settings" to activate your changes.

#1

Status:active» patch (code needs review)

Found the problem. It is extremely wierd. It had to do with the FOR EACH not specifying the LDBName while the fields in the PUBLISH actually did include the LDBName. Changes can be found in the PublishResults internal procedure.

Jurjen: I leave it to you how to manage this in the repository and release schedule. I have attached findsessions.w as findsessions.txt.


AttachmentSize
findsessions.txt16.09 KB
jurjen's picture

#2

Status:patch (code needs review)» closed

That is certainly weird - good thing you've found it! It is checked in into subversion now. Two things are scary here. One: what went wrong, this used to work and the code has not changed. Did it break from some OpenEdge improvement? Two: this same coding style may be everywhere - Prolint ought to have a rule to test for consistent ldbname usage :-)

   for each tablename  /* without ldbname */  :
       publish "event" ( ldbname.tablename.fieldname ). /* with ldbname */
   end.

Or did you just have two prolintdb's connected ;-)
Anyway, the code is checked in.