Bug 10846 - Field marked as "can be empty" when it cannot
Summary: Field marked as "can be empty" when it cannot
Status: CLOSED FIXED
Alias: None
Product: IPFire
Classification: Unclassified
Component: --- (show other bugs)
Version: 2
Hardware: unspecified Unspecified
: - Unknown - Balancing
Assignee: Assigned to nobody - feel free to grab it and work on it
QA Contact:
URL:
Keywords:
Depends on:
Blocks: 10847
  Show dependency treegraph
 
Reported: 2015-05-13 12:59 UTC by Larsen
Modified: 2015-12-22 16:30 UTC (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Larsen 2015-05-13 12:59:36 UTC
When you create a new roadwarrior connection, there is the field "Name der Organisation" which is marked with a star and thus should be possible to be left empty.

Though, on saving you will get the error "Ungültige Eingabe für Organisation". So, the star should be removed.

btw: Every other webpage or service I know uses the other way "items with a star need to be filled in". Care to change this behaviour?
Comment 1 Michael Tremer 2015-05-13 15:43:16 UTC
(In reply to Larsen from comment #0)
> When you create a new roadwarrior connection, there is the field "Name der
> Organisation" which is marked with a star and thus should be possible to be
> left empty.
> 
> Though, on saving you will get the error "Ungültige Eingabe für
> Organisation". So, the star should be removed.

This is not fully intentional. It should be possible to leave the field empty although it usually makes sense to copy that information from the CA certificate.

I guess I can confirm that this is a bug then.

> btw: Every other webpage or service I know uses the other way "items with a
> star need to be filled in". Care to change this behaviour?

I also regard this as a bug now. When the web interface has been written the default on the web was to declare the optional input fields optional. Nowadays it is the other way round and people do not check the bottom of the page any more to find out what the star is supposed to tell. They just assume that this is a required field. I do care to change this. This would just need to be changed on the entire web user interface at once for consistency. Would you like doing that? (This should be an extra bug report though)
Comment 2 Larsen 2015-05-13 16:16:50 UTC
(In reply to Michael Tremer from comment #1)
> assume that this is a required field. I do care to change this. This would
> just need to be changed on the entire web user interface at once for
> consistency. Would you like doing that? (This should be an extra bug report
> though)

Yes, I would.

http://wiki.ipfire.org/en/development/git is still valid?
Can you please provide me with a starting point, i.e. one small file where I can find the star?

IIRC, as an bugzilla admin you could clone this bug to separate it, can´t you?
Comment 3 Michael Tremer 2015-05-14 16:44:17 UTC
A better starting point would be this: http://wiki.ipfire.org/devel/start

If you clone the repository, you will find all the CGI files in html/cgi-bin which you will need to change to that the star is shown for the right inputs. The star image is somewhere in html/html/...
Comment 4 Larsen 2015-10-10 23:26:34 UTC
Uploaded a patch:
http://patchwork.ipfire.org/patch/100/
Comment 5 Larsen 2015-12-22 16:30:55 UTC
Has already been released with Core Update 94