Skip to main content

Please review my fix for CR 6595264

4 replies [Last post]
ersh
Offline
Joined: 2006-10-18

Show message if no conflicts/updates are found when "Check for template updates" is selected

- tester opens WD with config
- while tester has this WD opened, Admins updates template
- tester can select "Check for templates updates" to have changes propagated down to there config
- if there are no changes to template, and tester select "Check for templates updates" then there should be a simple message
--------------------
Please review my fix - https://jtharness.dev.java.net/source/browse/jtharness?view=rev&rev=1564

TIA,

Mike

Reply viewing options

Select your preferred way to display the comments and click "Save settings" to activate your changes.
kromanovs
Offline
Joined: 2007-11-14

Please consider the case when client code overrides InterviewParameters.load(*) methods. Previously they were all "void" but now "boolean". This is incompatible changes.

ersh
Offline
Joined: 2006-10-18

My fault. I didn't take into account that this class can be subclassed.

sergey_borodin
Offline
Joined: 2006-10-20

I'm agree,
please integrate.

Sergey

ersh
Offline
Joined: 2006-10-18

Thanks, Sergey.

Integrated