--- a/src/modules/client/image.py	Thu Apr 10 18:24:14 2008 -0700
+++ b/src/modules/client/image.py	Fri Apr 11 10:45:04 2008 -0700
@@ -937,7 +937,7 @@
                         self.fmri_set_default_authority(myfmri)
 
                 ofmri = self.optional_dependencies.get(name, None)
-                if not ofmri or self.fmri_is_successor(ofmri, myfmri):
+                if not ofmri or self.fmri_is_successor(myfmri, ofmri):
                                self.optional_dependencies[name] = myfmri
 
         def apply_optional_dependencies(self, myfmri):
@@ -955,7 +955,7 @@
                 if not minfmri:
                         return myfmri
 
-                if not myfmri.has_version() or self.fmri_is_successor(minfmri, myfmri):
+                if self.fmri_is_successor(minfmri, myfmri):
                         return minfmri
                 return myfmri