fix(select): throw Error that hide && submenu can not work

README.md

@@ -166,6 +166,7 @@ Can only be used when `isSetFunc` is not used.
 see `isSetFunc`
 
 `hide(ctx, key)` (optional) can be used to hide the button with the given key in the menu when true is returned.
+Can not be used when `submenu` is set.
 
 `columns` (Integer, optional) can be provided in order to limit the amount of buttons in one row. (default: 6)
 

inline-menu.js

@@ -275,13 +275,20 @@ class TelegrafInlineMenu {
     if (setFunc && submenu) {
       throw new Error('setFunc and submenu can not be set at the same time.')
     }
+    if (submenu && hide) {
+      // The submenu is a middleware that can do other things than callback_data
+      // question is an example: the reply to the text does not indicate the actionCode.
+      // Without the exact actionCode its not possible to determine the selected key(s) which is needed for hide(…, key)
+      throw new Error('hiding a dynamic submenu is not possible')
+    }
 
     const keyFromCtx = ctx => ctx.match[ctx.match.length - 1]
-    const hideSelectAction = hide && (ctx => hide(ctx, keyFromCtx(ctx)))
-
     const handler = {
-      action: new ActionCode(new RegExp(`${action}-([^:]+)`)),
-      hide: hideSelectAction
+      action: new ActionCode(new RegExp(`${action}-([^:]+)`))
+    }
+
+    if (additionalArgs.hide) {
+      handler.hide = ctx => hide(ctx, keyFromCtx(ctx))
     }
 
     if (setFunc) {

test/select.submenu.js

@@ -79,3 +79,14 @@ test('submenu button works', async t => {
 
   await bot.handleUpdate({callback_query: {data: 'a:c-a:d'}})
 })
+
+test('hide dynamic submenu does not work', t => {
+  const menu = new TelegrafInlineMenu('foo')
+
+  t.throws(() => {
+    menu.select('a', ['a', 'b'], {
+      hide: () => false,
+      submenu: new TelegrafInlineMenu('bar')
+    })
+  }, /dynamic/)
+})