#2651 closed defect (bug) (fixed)
Plugin Directory: Search result listing portion of full description instead of short description
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | Plugin Directory v3.0 | Priority: | high |
Component: | Plugin Directory | Keywords: | has-patch |
Cc: |
Description
May be related to #2585 but I just did a search for 'rating' and one plugin showed a portion of the full description instead of the short description from the readme.txt. Refreshed a few minutes later and it was back to normal (no changes made to the plugin).
Attachments (4)
Change History (14)
#1
@
8 years ago
- Component changed from Theme Directory to Plugin Directory
- Milestone Plugin Directory v3.0 deleted
#2
@
8 years ago
- Owner set to tellyworth
- Status changed from new to assigned
The second screenshot is from wp-admin, the first is from the web site.
I think this is either a bug in readme parsing, or something quirky in your readme.txt. Investigating.
@
8 years ago
Fix a bug in the readme parser class where blank lines were sometimes ignored following the headers.
#3
@
8 years ago
- Keywords has-patch added
Yup, seems it was a readme parser bug triggered by the colon in your short description.
For future testing, here's the readme content to repro:
=== Rating-Widget: Star Review System === Contributors: ratingwidget, svovaf, leorw, freemius Donate link: http://rating-widget.com/pricing/wordpress/ Tags: rate, rating, ratings, vote, votes, voting, star, stars, like, dislike, thumbs, widget, widgets, star rating, thumb rating, rating platform, rating system, seo, post rating, comment rating, 5 star, five star, sidebar, comment, comments, post, posts, page, pages, google, admin, administrator, review, reviews, rich-snippets, plugin, plugins, ajax, buddypress, bbpress, social, javascript, js, wordpress, meta, woocommerce, jquery, contest, competition, voting contest, rtmedia Requires at least: 3.0 Tested up to: 4.7 Stable tag: 2.8.6 License: GPLv2 or later The most popular Five Star Review System on the web. User-friendly ratings for: posts, pages, comments, WooCommerce, BuddyPress and bbPress forums. == Description == #### Ratings for your: * Post Ratings * Page Ratings
Seems to be faulty logic in the parser's handling of long header lines.
#4
@
8 years ago
As a note both screenshots were from wp.org, not one from /wp-admin and the other from wp.org. If it was from my wp-admin, it would have a "Compatible with..." part in the bottom right. Caching issue with the template rendering?
#7
follow-up:
↓ 8
@
8 years ago
- Resolution set to fixed
- Status changed from assigned to closed
In 5565:
#8
in reply to:
↑ 7
@
8 years ago
Replying to Otto42:
In 5565:
Just to note it here, IIRC me and @tellyworth chatted about that fix not being very complete.
There were a bunch of historical readme's which had extra line breaks between each header item, such as things like:
Some plugin name =============== Tested Up To: Something Stable Tag: 123 Tags: blah, blah blah Version: 1.2.3 Licenced under the GPL v2+ or later My Actual Short: Description
I'm totally happy for those readme's to break if they ever get re-parsed, but it's worth documenting here. Some may not have been historic even, and just poorly created text files.
Another "better" fix would be to only break if it was a non-header preceded by a blank line.
#10
@
8 years ago
Another "better" fix would be to only break if it was a non-header preceded by a blank line.
The problem is that it's considering any line with a colon in it to be a "header", because many plugins have non-standard headers there that we know not about.
I'm okay with "breaking" those plugins if they include extra useless blank lines in their headers. They can remove the blank lines and make the file look normal. But not having the short description appear or even get saved properly in the DB just because it contains a colon is kind of a bigger problem.
Shown the first time I searched, incorrectly showing portion of the full description