diff options
author | Svetoslav Ganov <svetoslavganov@google.com> | 2013-10-04 08:49:03 -0700 |
---|---|---|
committer | Svetoslav Ganov <svetoslavganov@google.com> | 2013-10-04 08:49:32 -0700 |
commit | 2b40c83ae1ec17ea9371c3fd3ac6c79c156faa1d (patch) | |
tree | db52b2ddc0066f3403f00d3593ea5915fe44a8d8 /packages/PrintSpooler | |
parent | 3d61bf45e559e084fa173bd50e2e7d7286dc9a35 (diff) | |
download | frameworks_base-2b40c83ae1ec17ea9371c3fd3ac6c79c156faa1d.zip frameworks_base-2b40c83ae1ec17ea9371c3fd3ac6c79c156faa1d.tar.gz frameworks_base-2b40c83ae1ec17ea9371c3fd3ac6c79c156faa1d.tar.bz2 |
Deleting print job files when we do not have to.
The print job file name was not properly parsed. This resulted
in incorrect mapping from file to print job, thus deleting the
file when we do not have to.
bug:11069354
Change-Id: I7b8a7a7f98dd77b34119f2a9cd16a6ec3c22b63b
Diffstat (limited to 'packages/PrintSpooler')
-rw-r--r-- | packages/PrintSpooler/src/com/android/printspooler/PrintSpoolerService.java | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/packages/PrintSpooler/src/com/android/printspooler/PrintSpoolerService.java b/packages/PrintSpooler/src/com/android/printspooler/PrintSpoolerService.java index d1d71cd..d1b42bc 100644 --- a/packages/PrintSpooler/src/com/android/printspooler/PrintSpoolerService.java +++ b/packages/PrintSpooler/src/com/android/printspooler/PrintSpoolerService.java @@ -440,8 +440,9 @@ public final class PrintSpoolerService extends Service { if (fileForJobMap == null) { fileForJobMap = new ArrayMap<PrintJobId, File>(); } - String printJobIdString = file.getName().substring(0, - PRINT_JOB_FILE_PREFIX.length()); + String printJobIdString = file.getName().substring( + PRINT_JOB_FILE_PREFIX.length(), + file.getName().indexOf('.')); PrintJobId printJobId = PrintJobId.unflattenFromString( printJobIdString); fileForJobMap.put(printJobId, file); |