Skip to content

Abstract duplicate code to retrieve a plugin information link#374

Merged
GaryJones merged 2 commits intodevelopfrom
feature/abstract-plugin-info-url
Apr 28, 2015
Merged

Abstract duplicate code to retrieve a plugin information link#374
GaryJones merged 2 commits intodevelopfrom
feature/abstract-plugin-info-url

Conversation

@jrfnl
Copy link
Copy Markdown
Contributor

@jrfnl jrfnl commented Apr 28, 2015

No description provided.

@jrfnl jrfnl added this to the 2.5.0 milestone Apr 28, 2015
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this one named with "plugin", yet the get_download_link() method isn't? Both are talking about plugins, so some consistency either way would be nice.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because the WP api for this calls it plugin information...

@jrfnl
Copy link
Copy Markdown
Contributor Author

jrfnl commented Apr 28, 2015

PR updated

@jrfnl jrfnl force-pushed the feature/abstract-plugin-info-url branch from eb41dab to 3aee995 Compare April 28, 2015 18:30
GaryJones added a commit that referenced this pull request Apr 28, 2015
…fo-url

Abstract duplicate code to retrieve a plugin information link
@GaryJones GaryJones merged commit 3569613 into develop Apr 28, 2015
@GaryJones GaryJones deleted the feature/abstract-plugin-info-url branch April 28, 2015 18:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants