Follow me on twitter

Ticket #297 (closed defect: fixed)

Opened 4 years ago

Last modified 10 months ago

partyline logging for alias cmds

Reported by: sknake Owned by: bryan
Priority: trivial Milestone: 1.3
Component: logging Version: 1.2.11
Keywords: Cc:

Description

The output isn't very useful:

on the leaf bot 

[22:44] <warjest> .bc
[22:44] <jeep> 'bc' is an invalid alias: references alias 'botcmd'.
[22:44] <jeep> Invalid alias 'bc' attempted: references alias 'botcmd'.

on the hub bot 

[22:44] <jeep> [02:42] (jeep) Invalid alias 'bc' attempted: references alias 'botcmd'.

Change History

  Changed 4 years ago by bryan 

show me output from .set list alias

follow-up: ↓ 6   Changed 4 years ago by sknake

well the problem is that on a leaf that .botcmd isnt a valid command, and .bc is set to botcmd. I just think it should include the username of the failed command on that line it logs to the hub.

[14:45] <usual> (list ) alias : bc botcmd,bl botcmd ?,+list set +,-list set -,list set list,aboot about,bd bc disc

  Changed 4 years ago by bryan 

This is more of a bug. It should display 'bc unknown cmd' on the leaf

  Changed 4 years ago by sknake 

That too, as long as it displays a username I'm happy

  Changed 4 years ago by bryan 

I forget what it looks like but it would be something like

! #bryan# botcmd
or
! #bryan# bc

Will have to think about it.

in reply to: ↑ 2   Changed 4 years ago by plat0nic

Replying to sknake:

well the problem is that on a leaf that .botcmd isnt a valid command, and .bc is set to botcmd. I just think it should include the username of the failed command on that line it logs to the hub. [14:45] <usual> (list ) alias : bc botcmd,bl botcmd ?,+list set +,-list set -,list set list,aboot about,bd bc disc

".aboot" lol, I saw that and busted up over it. That's great.

  Changed 3 years ago by bryan

  • summary changed from partyline logging to partyline logging for alias cmds

  Changed 10 months ago by bryan

  • milestone set to 1.2.17

  Changed 10 months ago by bryan

Amazing, just found this one today myself. Come back scott!

  Changed 10 months ago by Bryan Drewery

  • status changed from new to closed
  • resolution set to fixed

* Fix aliases not properly indicating bad command on leaf bots (fixes #297)

Changeset: ae0b2ac4960855c69aeb3dcdbc70194d696747ac

  Changed 10 months ago by Bryan Drewery

* Fix aliases not properly indicating bad command on leaf bots (fixes #297)

Changeset: ae0b2ac4960855c69aeb3dcdbc70194d696747ac

  Changed 10 months ago by Bryan Drewery

Merge branch '268-whitespace' into next

* 268-whitespace:

  • Cleanup all string parsing to trim excess whitespaces (fixes #268)
  • Fix aliases not properly indicating bad command on leaf bots (fixes #297)
  • Add warning againt using cmd_botjoin

Conflicts:

doc/UPDATES

Changeset: 7eea226bf5993703202e429774f7bd2652c2d81c

Note: See TracTickets for help on using tickets.