Skip to content
This repository has been archived by the owner on Jan 2, 2019. It is now read-only.

Fix wrong logic in VLOOKUP #453

Closed
wants to merge 1 commit into from
Closed

Fix wrong logic in VLOOKUP #453

wants to merge 1 commit into from

Conversation

jumoel
Copy link

@jumoel jumoel commented Oct 30, 2014

Before:

  • If lookup value was numeric, the return value also had to be numeric.
  • If the lookup value wasn't numeric, the return value also couldn't be numeric.

This doesn't make sense. What is returned does not and should not depend in any way on the type of what is being looked for. The documentation also doesn't mention anything about this.

The behaviour of VLOOKUP is now also matching the behaviour of HLOOKUP.

See also #385.

Before:
 - If lookup value was numeric, the return value also had to be numeric.
 - If the lookup value wasn't numeric, the return value also couldn't be numeric.

This doesn't make sense. What is returned does not and should not depend in any way on the type of what is being looked for. The [documentation](http://office.microsoft.com/en-us/excel-help/vlookup-HP005209335.aspx) also doesn't mention anything about this.

The behaviour of `VLOOKUP` is now also matching the behaviour of `HLOOKUP`.
@jumoel
Copy link
Author

jumoel commented Oct 30, 2014

Hadn't seen #423, which is the exact same fix.

@jumoel jumoel closed this Oct 30, 2014
@jumoel jumoel deleted the patch-2 branch October 31, 2014 06:34
@Progi1984 Progi1984 added this to the 1.8.1 milestone Oct 31, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants