(temp-directory) needs some (regexp-quote) love in gnuserv.el

Stephen J. Turnbull stephen at xemacs.org
Thu Feb 14 22:38:23 EST 2008


Added as issue292 in the tracker.

skip at pobox.com writes:

 >     >> Error: Required issue property module not supplied
 > 
 >     Stephen> Couldn't you select one?  Or did you just ignore it?
 > 
 > A quick glance around the screen didn't reveal anything.  What is
 > an "issue property module"?  The error message read as if there was
 > a missing Python module, not that I forgot to fill out some
 > required field.

Ah, I see. Your problem is you know too much about the implementation
for your own good.<wink>

What you see on screen is the display of an "issue", which is actually
a Python object.  In roundup-speak, the attributes of the object are
called "properties".  If you look at the screen you'll see the place
for entering the "issue property" called "Module" at the upper right,
same row as "Type".  Unfortunately, the internal name of the attribute
is lowercase, leading to that misleading error message.

I'll put quotes around the property name or something.

 > A better user interface would also highlight the title of the field
 > which wasn't filled in (maybe render it in red text).

I'll see what I can do.  Does Python's do that?  If so I can adapt
MvL's code. :-)

 > Also, now that I go back and study the submission form more
 > closely, I see that the explanation of the title highlighting is at
 > the bottom of the form.  Shouldn't that be at the top?

Well, I was hoping it wouldn't be needed, and actually removed it from
the editing view.  I didn't realize the new issue view was different,
so it's still there.  (Before I implemented the Module property, it
*wasn't* needed, everything else that's "required" has a reasonable
default.)

For the new issue view, my Mozilla requires that much vertical space
for the sidebar (mine's probably taller than yours because of saved
queries), and there are no messages yet so vertical space is not at a
premium.  I'll do as you suggest, which is better than it is now.  I
think I'll also move the submit button up there and put the "Process"
box at the very bottom.



More information about the XEmacs-Beta mailing list