SHIMADA Koji
null+****@clear*****
Fri Oct 26 20:33:26 JST 2012
SHIMADA Koji 2012-10-26 20:33:26 +0900 (Fri, 26 Oct 2012) New Revision: 9d98eab3f84b88bf612575fc77ffa96a38bfdee4 https://github.com/logaling/logaling-server/commit/9d98eab3f84b88bf612575fc77ffa96a38bfdee4 Merged 5645d73: Merge pull request #38 from logaling/fix-empty-search Log: Show guidance when accessing '/search' in direct with no parameter Added files: features/step_definitions/logaling_server_steps.rb Modified files: app/controllers/search_controller.rb app/views/search/index.html.haml features/search_term.feature Modified: app/controllers/search_controller.rb (+1 -1) =================================================================== --- app/controllers/search_controller.rb 2012-10-19 09:43:18 +0900 (d91b28b) +++ app/controllers/search_controller.rb 2012-10-26 20:33:26 +0900 (b9a4e7f) @@ -1,6 +1,6 @@ class SearchController < ApplicationController def index - @query = params[:query] + @query = params[:query] || '' priority_glossary = signed_in? ? current_user.priority_glossary : nil search_results = LogalingServer.repository.lookup(@query, priority_glossary).map do |t| Modified: app/views/search/index.html.haml (+17 -14) =================================================================== --- app/views/search/index.html.haml 2012-10-19 09:43:18 +0900 (bf1f17a) +++ app/views/search/index.html.haml 2012-10-26 20:33:26 +0900 (25f539a) @@ -2,18 +2,21 @@ %div.container %h1 = @title - - if****@terms*****? - %p - = count_of_starting_position(@per_count, @terms.current_page) - \- - = count_of_end_position(@per_count, @terms.current_page, @terms.count) - 件表示 / - =****@terms*****_count - 件中 - -****@terms***** do |term| - = render :partial => 'shared/term', :locals => {:term => term} - = paginate @terms + - if****@query*****? + - if****@terms*****? + %p + = count_of_starting_position(@per_count, @terms.current_page) + \- + = count_of_end_position(@per_count, @terms.current_page, @terms.count) + 件表示 / + =****@terms*****_count + 件中 + -****@terms***** do |term| + = render :partial => 'shared/term', :locals => {:term => term} + = paginate @terms + - else + %p + = @query + は見つかりませんでした。 - else - %p - = @query - は見つかりませんでした。 + %p 単語/フレーズを入力して検索してください Modified: features/search_term.feature (+15 -0) =================================================================== --- features/search_term.feature 2012-10-19 09:43:18 +0900 (591d079) +++ features/search_term.feature 2012-10-26 20:33:26 +0900 (8bcff28) @@ -10,3 +10,18 @@ かつ トップページを表示している もし "logaling server"と検索する ならば "logaling サーバ"と表示されていること + + シナリオ: 登録されていない用語の検索 + 前提 "logaling"ユーザの"logaling-server"プロジェクトが登録済みである + かつ トップページを表示している + もし "groonga"と検索する + ならば "groonga は見つかりませんでした。"と表示されていること + + シナリオ: 検索ワードに何も入力せずに検索ボタンをクリックした場合 + 前提 トップページを表示している + もし ""と検索する + ならば "単語/フレーズを入力して検索してください"と表示されていること + + シナリオ: /search にパラメータを付けずに直接アクセスした場合 + もし "/search"にパラメータを付けずに直接アクセスする + ならば "単語/フレーズを入力して検索してください"と表示されていること Added: features/step_definitions/logaling_server_steps.rb (+5 -0) 100644 =================================================================== --- /dev/null +++ features/step_definitions/logaling_server_steps.rb 2012-10-26 20:33:26 +0900 (fb7b9f9) @@ -0,0 +1,5 @@ +#coding: utf-8 + +もし /^"([^"]*)"にパラメータを付けずに直接アクセスする$/ do |path| + visit path +end -------------- next part -------------- An HTML attachment was scrubbed...ダウンロード
