The Inbox: MonticelloConfigurations-dtl.133.mcz

David T. Lewis uploaded a new version of MonticelloConfigurations to project The Inbox:
http://source.squeak.org/inbox/MonticelloConfigurations-dtl.133.mcz

==================== Summary ====================

Name: MonticelloConfigurations-dtl.133
Author: dtl
Time: 10 May 2015, 7:38:29.29 pm
UUID: d5d2f00e-20d2-4e1e-bb26-6f087e3d9c28
Ancestors: MonticelloConfigurations-dtl.132

MCMcmUpdater>>refreshUpdateMapFor:with: should not answer an empty update list in the case of a package that has not yet been loaded. Fixes an existing bug, not related to the refactoring in the last two updates.

=============== Diff against MonticelloConfigurations-dtl.132 ===============

Item was changed:
  ----- Method: MCMcmUpdater>>refreshUpdateMapFor:with: (in category 'updating') -----
  refreshUpdateMapFor: r with: updateList
  "Update the lastUpdateMap and answer a possibly reduced updateList"
 
  | config |
  (lastUpdateMap at: r description ifAbsent: [0]) = 0 ifTrue: [
  "No update has ever been loaded from this repo. If no package is
  present in the image either, we can skip right to the latest config"
  config := r versionNamed: updateList last value.
+ (config dependencies anySatisfy: [:dep | dep package hasWorkingCopy])
+ ifFalse: [(self useLatestPackagesFrom: r)
+ ifTrue: [lastUpdateMap at: r description put: updateList last key].
+ updateList isEmpty
+ ifTrue: [^ #()]
+ ifFalse: [^ updateList last: 1]]].
- (config dependencies anySatisfy: [:dep| dep package hasWorkingCopy])
- ifFalse: [ (self useLatestPackagesFrom: r)
- ifTrue: [lastUpdateMap at: r description put: updateList last key.
- ^ #()]
- ifFalse: [ ^ updateList last: 1]]].
  ^ updateList
  !