Menu

#37 Label containing more than one word

open
nobody
None
5
2007-08-05
2007-08-05
firmit
No

Adding labels with more than one word creates wrong XML in prefs.xml

"<tagname lblFIRSTword lblSECONDword="60" />"

Easy fixed I guess...

Discussion

  • firmit

    firmit - 2007-08-05

    Logged In: YES
    user_id=1860768
    Originator: YES

    naturally caused checkgmail NOT to start....

    ps: my first bug-report in sourceforge

     
  • Owen Marshall

    Owen Marshall - 2007-08-07

    Logged In: YES
    user_id=874762
    Originator: NO

    Not so easily fixed, I fear. (I think I might have to store the data structure as the multi-dimensional array that I use for the preferences, but getting that into xml::simple hasn't been as easy as I'd like ...) If you've got a suggestion as to how to get xml::simple to give a correctly formatted file I'm all ears :)

    Otherwise, I'll work on it when I've got the time (which isn't now!)

    Cheers,
    Owen

     
  • firmit

    firmit - 2007-08-07

    Logged In: YES
    user_id=1860768
    Originator: YES

    Hmm - why not change the structure of the xml line containing the label?
    A new label generates the xml: "<label_delay ONEWORD="300" />

    Problem is when there is multiword-label: "<label_delay BBO forum="300" />"

    Why not change the structure to: "<label delay="300">Label name</label>" or "<label name="label name" delay="300"/>

    Is this not what you mean when you say xml::simple? Or do you want everything to be in one tag as attributes?

     
  • firmit

    firmit - 2007-08-07

    Logged In: YES
    user_id=1860768
    Originator: YES

    Actually - there is a work-around.

    I checked in GMail - and when you chose a label, gmail search is: "label:firstword-secondword"

    When I changed the label to "BBO-forum" it worked. Hence, when adding a label to the prefs.xml, just explode " " with "-". And when displaying the prefs GUI, do it the other way around....

    "<label_delay BBO-forum="300"/>" is allowed

     
  • Owen Marshall

    Owen Marshall - 2007-08-07

    Logged In: YES
    user_id=874762
    Originator: NO

    Ah, OK, that makes it a lot simpler. I'll try to implement it when I get the chance :)

    Cheers,
    Owen

     
  • Nobody/Anonymous

    Logged In: NO

    Also label starting with special chars gives the same problem.
    Eg.:

    a label called: @mylabel

    in prefs.xml becomes:

    <label_delay @mylabel="300"/>

     

Log in to post a comment.