View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0000195 | AniDB IRC (Chii/Lafiel) | Bug Report | public | 2005-06-03 20:39 | 2008-02-25 08:28 |
Reporter | Rar | Assigned To | pelican | ||
Priority | normal | Severity | trivial | Reproducibility | always |
Status | resolved | Resolution | fixed | ||
Summary | 0000195: Make !votetmp report itself as VOTETMP not VOTE | ||||
Description | In all chii responses, the !votetmp command suffers from copycode effect, and still thinks it's a !vote. <rar> !votetmp <Chii[AR]> VOTE: usage: !vote <aid> [1-10], !vote <anime title> [1-10] | ||||
Additional Information | Probably safe to do find/replace "3VOTE:" with "3VOTETMP:" in the relevant file. | ||||
Tags | No tags attached. | ||||
|
it's actually not a copy&paste bug, both commands use the same codebase. |
|
Declare the $out variable higher up, and set it dependant on the command given? Hard to fix without seeing the code, but: my $out = $main::color."4VOTE:" / my $out = $main::color."4VOTETMP:" / my $out = $main::color."4VOTEG:" Should all be setable near the beginning based on the type of the command, then any common subfunctions you use that return output don't need to print the head part at all, and you can just append what they give: $out .= //common subfunc for reporting existing vote |
|
fixed by pelican |
Date Modified | Username | Field | Change |
---|---|---|---|
2005-06-03 20:39 | Rar | New Issue | |
2005-06-09 20:40 | exp | Note Added: 0000514 | |
2005-06-10 07:39 | Rar | Note Added: 0000522 | |
2005-07-06 20:24 | exp | Status | new => resolved |
2005-07-06 20:24 | exp | Resolution | open => fixed |
2005-07-06 20:24 | exp | Assigned To | => exp |
2005-07-06 20:24 | exp | Note Added: 0000580 | |
2005-07-06 20:24 | exp | Assigned To | exp => |
2008-02-25 08:28 | epoximator | Assigned To | => pelican |