When searching for a story by URL, the parse tree displayed for the search query omits that URL

[roryokane]

Bug description

When searching for a story by URL, such as https://lobste.rs/search?q=https%3A%2F%2Fghuntley.com%2Ffracture%2F&what=stories&order=newest, after expanding the “n results for” section, the parse tree that should be below “Search query parsed to:” is empty:

This empty parse tree is visible by default when there are no results:

For comparison, search terms that aren’t URLs (as in this search) are described in that section:

The “Search query parsed to:” section should describe how the URL was parsed from the search, too.

Implementation suggestions

The search parse tree is displayed in this view:

lobsters/app/views/search/index.html.erb

Lines 53 to 62 in c7ef797

	<p>
	Search query parsed to:
	</p>
	<dl class="search_parse">
	<%
	@search.parse_tree.each do |node|
	type, value = node.first
	case type
	when :commenter %>

That case statement needs a when :url branch.

The chain of code that shows the parse tree should contain a :url key with the URL as its value

lobsters/app/controllers/search_controller.rb

Line 10 in c7ef797

@search = Search.new(search_params, @user)

lobsters/app/models/search.rb

Lines 18 to 19 in c7ef797

	@parse_tree = if params[:q].present?
	SearchParser.new.parse(params[:q])

lobsters/app/models/search_parser.rb

Line 52 in c7ef797

root(:expression)

lobsters/app/models/search_parser.rb

Line 42 in c7ef797

url |

lobsters/app/models/search_parser.rb

Lines 22 to 27 in c7ef797

	rule(:url) {
	(
	str("http") >> str("s").repeat(0, 1) >> str("://") >>
	match("[A-Za-z0-9\\-_.:@/()%~?&=#]").repeat(1)
	).as(:url) >> space?
	}

.as(:url) ensures the resulting parse tree Hash will contain a :url key. See as described in Parslet’s parsing docs.

The ERB inside the new when :url branch should represent a dt element followed by a dd element, the same as all the other branches.