Project: Eclipse_Platform_UI
Bug 307901 EPartService does not short circuit saveAll(boolean) if 'false' and/or no save handler is provided
Reported: 2010-04-01 12:48:49 | Status: resolved fixed

The code looks like so... boolean success = true; for (MPart dirtyPart : dirtyParts) { if (!savePart(dirtyPart, false)) { success = false; } } return success; ...the first time a save fails, we should return false immediately.

Files changed in fix:
bundles/org.eclipse.e4.ui.workbench/src/org/eclipse/e4/workbench/ui/internal/PartServiceImpl.java tests/org.eclipse.e4.ui.tests/src/org/eclipse/e4/ui/tests/application/EPartServiceTest.java