Fix hashtag search performing account search as well (#13758)
This commit is contained in:
		
							parent
							
								
									60bb6ac6f4
								
							
						
					
					
						commit
						063940f489
					
				
					 2 changed files with 9 additions and 1 deletions
				
			
		| 
						 | 
					@ -94,7 +94,7 @@ class SearchService < BaseService
 | 
				
			||||||
  end
 | 
					  end
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  def account_searchable?
 | 
					  def account_searchable?
 | 
				
			||||||
    account_search? && !(@query.include?('@') && @query.include?(' '))
 | 
					    account_search? && !(@query.start_with?('#') || (@query.include?('@') && @query.include?(' ')))
 | 
				
			||||||
  end
 | 
					  end
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  def hashtag_searchable?
 | 
					  def hashtag_searchable?
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -91,6 +91,14 @@ describe SearchService, type: :service do
 | 
				
			||||||
          expect(Tag).not_to have_received(:search_for)
 | 
					          expect(Tag).not_to have_received(:search_for)
 | 
				
			||||||
          expect(results).to eq empty_results
 | 
					          expect(results).to eq empty_results
 | 
				
			||||||
        end
 | 
					        end
 | 
				
			||||||
 | 
					        it 'does not include account when starts with # character' do
 | 
				
			||||||
 | 
					          query = '#tag'
 | 
				
			||||||
 | 
					          allow(AccountSearchService).to receive(:new)
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					          results = subject.call(query, nil, 10)
 | 
				
			||||||
 | 
					          expect(AccountSearchService).to_not have_received(:new)
 | 
				
			||||||
 | 
					          expect(results).to eq empty_results
 | 
				
			||||||
 | 
					        end
 | 
				
			||||||
      end
 | 
					      end
 | 
				
			||||||
    end
 | 
					    end
 | 
				
			||||||
  end
 | 
					  end
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
		Reference in a new issue