[Chii] !addstitle takes only first word [FIXED]

[wahaha]

Code: Select all

<wahaha> !addstitle 219 CB Movie
<Chii[AR]> ADDSTITLE: Cowboy Bebop - Knockin` on heaven`s door: added short title - CB.

Well, this is not what I intended to do ...

Plus, it obviously rejected that shortitle since "CB" was already in, it'd be nice to have a message that indicates that

[exp]

well,

fixed

although maybe not in the way u thought about it.
- short titles may not contain spaces (added error message stating that)
- existing titles are not allowed as short titles (added new error message for that)

BYe!
EXP

[wahaha]

although maybe not in the way u thought about it.

How dare you

- short titles may not contain spaces (added error message stating that)

Why that?

Examples for *existing* short tiles (not synonyms) with spaces:
FY OVA2
GitS SAC
MTT TV / MTT OVA

Dunno if there are more, but these both need their spaces and *are* short-titles... OTOH, putting "GitS SAC" aside, Chii could instead automatically take the "Movie-" "TV-" or "OVA-"parts into account when trying to find the corresponding db-entry - that'd somehow be a real feature-req, though

[exp]

hm,

guess you're right, i will change !addstitle to allow spaces too once i get to it

the movie/ova/tv stuff, well we could think about that one, however only for the !anime command and there it would work like this.
- check if there is a single exact match (with movie/ova/.. appended)
if not
- check if there is a single exact match (with movie/ova/.. removed)
if not
- check if there are multiple matches (with movie/ova/.. removed)
if yes
- check if there is only one match left if we require type to be movie/ova/..
if we still don't have any match do the same stuff again but for partial matches.

as soon as we get one match, display it.
is that what u had in mind?

BYe!
EXP

[wahaha]

is that what u had in mind?

To be honest, I just wanted to tease you ... but what you wrote sounds good