From 5a729f66672d84f63c9f295008387d675a87b795 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Marc=20Cornell=C3=A0?= <marc.cornella@live.com>
Date: Sat, 15 Sep 2018 22:56:12 +0200
Subject: [PATCH] lib: fix history wrapper when passing numbers

If a number is passed without explicitly passing `-l`, it will now behave
as if using the history builtin, instead of throwing an error.
---
 lib/history.zsh | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/lib/history.zsh b/lib/history.zsh
index 62e02648b..d8bbd41c4 100644
--- a/lib/history.zsh
+++ b/lib/history.zsh
@@ -1,7 +1,6 @@
 ## History wrapper
 function omz_history {
-  # Delete the history file if `-c' argument provided.
-  # This won't affect the `history' command output until the next login.
+  local clear list
   zparseopts -E c=clear l=list
 
   if [[ -n "$clear" ]]; then
@@ -12,9 +11,8 @@ function omz_history {
     # if -l provided, run as if calling `fc' directly
     builtin fc "$@"
   else
-    # otherwise, call `fc -l 1` to show all available
-    # history (and pass additional parameters)
-    builtin fc "$@" -l 1
+    # unless a number is provided, show all history events (starting from 1)
+    [[ ${@[-1]} = *[0-9]* ]] && builtin fc -l "$@" || builtin fc -l "$@" 1
   fi
 }