Skip to content

Index hash_key values were empty because of the key used.#124

Merged
loganb merged 2 commits intojoshsymonds:masterfrom
stefanneculai:fix_index_hash_key
Jun 16, 2013
Merged

Index hash_key values were empty because of the key used.#124
loganb merged 2 commits intojoshsymonds:masterfrom
stefanneculai:fix_index_hash_key

Conversation

@stefanneculai
Copy link
Copy Markdown
Contributor

Index hash_key values were empty because of the key used. This was happening when the key used was email. I am not sure if it was because of @ or . character from the key value..

@stefanneculai
Copy link
Copy Markdown
Contributor Author

OK.. Writing the spec for this I find out that the problem was not the one I described above but the fact that sometimes hashes were using strings and sometimes symbols. I will commit shortly the correct fix + the spec for it.

@loganb
Copy link
Copy Markdown
Collaborator

loganb commented Jun 16, 2013

Great catch!

loganb added a commit that referenced this pull request Jun 16, 2013
Symbolize keys passed to Chain#where()
@loganb loganb merged commit 19a5cbd into joshsymonds:master Jun 16, 2013
@stefanneculai stefanneculai deleted the fix_index_hash_key branch June 20, 2013 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants