Bugzilla – Full Text Bug Listing
| Summary: | Extra information displayed when create patch failed | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | [Eclipse Project] Platform | Reporter: | Malgorzata Janczarska <malgorzata.tomczyk> | ||||||||
| Component: | CVS | Assignee: | Malgorzata Janczarska <malgorzata.tomczyk> | ||||||||
| Status: | RESOLVED FIXED | QA Contact: | |||||||||
| Severity: | normal | ||||||||||
| Priority: | P3 | CC: | daniel_megert, Szymon.Brandys, tomasz.zarna | ||||||||
| Version: | 3.7 | ||||||||||
| Target Milestone: | 4.3 M1 | ||||||||||
| Hardware: | All | ||||||||||
| OS: | All | ||||||||||
| Whiteboard: | |||||||||||
| Attachments: |
|
||||||||||
|
Description
Malgorzata Janczarska
Created attachment 193916 [details] Display information in the error dialog This patch moves the information message to the error dialog. See Bug 319661 comment #54 for remarks to this bug. Not critical enough to be addressed during RC*. Patch pushed to Gerrit: https://git.eclipse.org/r/#/c/5242/ Created attachment 212572 [details]
Old "No differences found" dialog
Created attachment 212573 [details]
New "No differences found" dialog
Please go to Gerrit: https://git.eclipse.org/r/#/c/5242/ for the new version of patch. I used StatusManager to handle displaying messages. This has two advantages: 1. The "No differences found" is not shown as an extra dialog. If there where errors while creating patch they are reported altogether in one error dialog. "No differences found" is also part of this dialog. 2. Previously if we where creating patch for 4 projects and all of them failed we used to get 4 error message dialogs. Now we get only one dialog with all 4 errors. But in result of this we have a visual change in reporting messages, even if we don't get any errors the "No differences found" info differs. See attachment 212572 [details] for the old version of the dialog and attachment 212573 [details] of the new version. I also changed it's title to "CVS diff is empty" as in the new version I think "CVS diff" is not sufficient. If this change is not acceptable I may create a simple "if" to avoid changing the "No differences found" if there where no errors. > Please go to Gerrit: https://git.eclipse.org/r/#/c/5242/ for the new version
> of patch.
Ping! Review has been made a while ago ;-).
(In reply to comment #7) > > Please go to Gerrit: https://git.eclipse.org/r/#/c/5242/ for the new version > > of patch. > > Ping! Review has been made a while ago ;-). Sorry, please give me some more time, I'm loaded with other also important work currently. (In reply to comment #8) > (In reply to comment #7) > > > Please go to Gerrit: https://git.eclipse.org/r/#/c/5242/ for the new version > > > of patch. > > > > Ping! Review has been made a while ago ;-). > > Sorry, please give me some more time, I'm loaded with other also important work > currently. The suggested adjustment seems to be easy to apply. Gosia, redo the patch and ask Dani for review again, please. I can see it has been merged in 492c044a64ed6b8a6a07fd7bbc4be551487791ac. Thanks Dani. (In reply to comment #10) > I can see it has been merged in 492c044a64ed6b8a6a07fd7bbc4be551487791ac. Is the target milestone correct? The change has been merged in on June 14, that would mean 3.8, right? (In reply to comment #11) > (In reply to comment #10) > > I can see it has been merged in 492c044a64ed6b8a6a07fd7bbc4be551487791ac. > > Is the target milestone correct? The change has been merged in on June 14, that > would mean 3.8, right? No, it's in 4.3. |