Update EditView.tpl #179

Open
wants to merge 1 commit into
from

Projects

None yet

3 participants

@eggsurplus
SugarCRM member

If the value is blank/null in the database and a record is edited the UI will show that the default value(s) are selected. This is not desired especially when there is a blank option for an enum or when a user deselects all options on a multienum. It creates what is an unexpected behavior for the user.

@eggsurplus eggsurplus Update EditView.tpl
If the value is blank/null in the database and a record is edited the UI will show that the default value(s) are selected. This is not desired especially when there is a blank option for an enum or when a user deselects all options on a multienum. It creates what is an unexpected behavior for the user.
00a4156
@Sugared

Hello eggsurplus,

Thanks for the pull request. Have you had a chance to file a bug for this issue? If not, can you please file a bug through our portal (http://bugs.sugarcrm.com/) and include the details on what problem is being addressed as well as how to reproduce this bug? It also helps us when you make reference to this pull request and include the url.

thanks.

@eggsurplus
SugarCRM member

Seems like a waste of a step when I explain the situation it addresses in the pull request comments above. I'd use the bugs site, but it is too painful to search for existing bugs for this problem.

The fix is straight forward. Only use the default enum value if it is a new record.

Just trying to help.

@jmertic

@eggsurplus we need a bug to track this for QA purposes; if you can't do this then we won't be able to pull in the fix.

@eggsurplus
SugarCRM member

Done as requested - http://www.sugarcrm.com/support/bugs.html#issue_65680

Understood, but it would be great if this could be done automatically as part of a pull request. It's just tedious to search for existing bugs (assuming that we should still do that) and then create a bug for each pull request. Not sure for others, but it has held me back from doing pull requests in the past.

@jmertic
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment