Minor bug fixes: #555

Closed
wants to merge 1 commit into from

4 participants

@pranavkm
NuGet member
  • Work Item #447: Use Published Date instead of last updated date in the package display
  • Store null values in the Download stats pag*
  • Work Item #465: Allow at most 5 tags per package
@pranavkm pranavkm Minor bug fixes:
* Work Item #447: Use Published Date instead of last updated date in the
  package display
* Store null values in the Download stats pag*
* Work Item #465: Allow at most 5 tags per package
68cb08e
@pranavkm pranavkm commented on the diff Jul 26, 2012
Website/Services/PackageService.cs
@@ -282,7 +282,11 @@ Package CreatePackageFromNuGetPackage(PackageRegistration packageRegistration, I
if (nugetPackage.Summary != null)
package.Summary = nugetPackage.Summary;
if (nugetPackage.Tags != null)
- package.Tags = nugetPackage.Tags;
+ {
+ // To prevent tag abuse, we'll allow at most 5 tags to be read.
+ var tags = nugetPackage.Tags.Split(' ');
+ package.Tags = String.Join(" ", tags.Take(5));
+ }
@pranavkm
NuGet member

Should probably make this 10.

@Haacked
NuGet member
Haacked added a note Jul 27, 2012

Agree.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
@jeffhandley
NuGet member

These fixes are in the 'development' branch and are ready for testing.

@davidfowl
NuGet member

Why is this PR still open?

@pranavkm pranavkm closed this Jan 21, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment