Fix ls -S to deal with broken/unparsable app versions more correctly #385
| @ -176,25 +176,28 @@ can take some time. | |||||||
| 					appStats.AutoUpdate = autoUpdate | 					appStats.AutoUpdate = autoUpdate | ||||||
|  |  | ||||||
| 					var newUpdates []string | 					var newUpdates []string | ||||||
| 					if version != "unknown" { |  | ||||||
| 						updates, err := recipe.GetRecipeCatalogueVersions(app.Recipe, catl) |  | ||||||
| 						if err != nil { |  | ||||||
| 							logrus.Fatal(err) |  | ||||||
| 						} |  | ||||||
|  |  | ||||||
|  | 					if (version != "unknown") { | ||||||
|  | |||||||
| 						parsedVersion, err := tagcmp.Parse(version) | 						parsedVersion, err := tagcmp.Parse(version) | ||||||
| 						if err != nil { | 						if (err != nil) { | ||||||
| 							logrus.Fatal(err) | 							logrus.Warning("Can't parse ", app.Recipe, " version ", version, " ", err) | ||||||
| 						} | 						} else { | ||||||
|  | 							updates, err := recipe.GetRecipeCatalogueVersions(app.Recipe, catl) | ||||||
|  |  | ||||||
| 						for _, update := range updates { |  | ||||||
| 							parsedUpdate, err := tagcmp.Parse(update) |  | ||||||
| 							if err != nil { | 							if err != nil { | ||||||
| 								logrus.Fatal(err) | 								logrus.Fatal(err) | ||||||
| 							} | 							} | ||||||
|  |  | ||||||
| 							if update != version && parsedUpdate.IsGreaterThan(parsedVersion) { | 							for _, update := range updates { | ||||||
| 								newUpdates = append(newUpdates, update) | 								parsedUpdate, err := tagcmp.Parse(update) | ||||||
|  | 								if err != nil { | ||||||
|  | 									logrus.Warning("can't parse ", app.Recipe," update version ", update, " ", err) | ||||||
|  | 									continue | ||||||
|  | 								} | ||||||
|  |  | ||||||
|  | 								if update != version && parsedUpdate.IsGreaterThan(parsedVersion) { | ||||||
|  | 									newUpdates = append(newUpdates, update) | ||||||
|  | 								} | ||||||
| 							} | 							} | ||||||
| 						} | 						} | ||||||
| 					} | 					} | ||||||
|  | |||||||
		Reference in New Issue
	
	Block a user
	
(...)not required?