Fix proposed in Issue NuGet#1627

Fixed the Trace.Activity for the line 101. Before is was accessing a null variable.
fabiostawinski · Oct 30, 2013 · eb33ec4 · eb33ec4
1 parent b291c4e
commit eb33ec4
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/NuGetGallery/Infrastructure/Lucene/LuceneIndexingService.cs b/src/NuGetGallery/Infrastructure/Lucene/LuceneIndexingService.cs
@@ -98,13 +98,13 @@ public void UpdatePackage(Package package)
  }
 
  // Just update the provided package
- using (Trace.Activity(String.Format(CultureInfo.CurrentCulture, "Updating Lucene Index for: {0} {1} [PackageKey:{2}]", id, version, key)))
+ using (Trace.Activity(String.Format(CultureInfo.CurrentCulture, "Updating Document: {0}", updateTerm.ToString())))
  {
  EnsureIndexWriter(creatingIndex: false);
  if (package != null)
  {
  var indexEntity = new PackageIndexEntity(package);
- Trace.Information(String.Format(CultureInfo.CurrentCulture, "Updating Document: {0}", updateTerm.ToString()));
+ Trace.Information(String.Format(CultureInfo.CurrentCulture, "Updating Lucene Index for: {0} {1} [PackageKey:{2}]", package.PackageRegistration.Id, package.Version, package.Key));
  _indexWriter.UpdateDocument(updateTerm, indexEntity.ToDocument());
  }
  else