Bug 32097 - umc/widgets/Text.isValid() missing
umc/widgets/Text.isValid() missing
Status: CLOSED DUPLICATE of bug 34484
Product: UCS
Classification: Unclassified
Component: UMC (Generic)
UCS 3.1
All Linux
: P5 enhancement (vote)
: UCS 3.2-2
Assigned To: Alexander Kläser
Florian Best
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-07-26 19:39 CEST by Philipp Hahn
Modified: 2014-11-11 15:01 CET (History)
2 users (show)

See Also:
What kind of report is it?: ---
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Philipp Hahn univentionstaff 2013-07-26 19:39:38 CEST
Uncaught TypeError: Object [object Object] has no method 'isValid' 

uvmm/DriveWizard.js:319
  valid = valid && (!iwidget.get('visible') || false !== iwidget.isValid());

It would be nice if a dummy "function isValid() { return true; }" would be defined for umc/widget/Text, since Text is sometimes used in a Form to display additional Text, for example in UVMM when the storage pool capacity is exceeded.

univention-management-console-frontend/umc/widgets/Text.js
Comment 1 Philipp Hahn univentionstaff 2013-07-26 21:03:08 CEST
svn32097: for now UVMM ignores a missing isValid().
Comment 2 Alexander Kläser univentionstaff 2013-07-30 16:01:15 CEST
It is also possible to use umc/widgets/Form:validate(), this method takes care of exactly that problem. Would that be of use?
Comment 3 Alexander Kläser univentionstaff 2014-07-11 12:45:33 CEST
... as I am working on validation issues anyway, I added the function.
Comment 4 Alexander Kläser univentionstaff 2014-07-21 17:13:32 CEST
This bug has been fixed along with Bug 34484.

*** This bug has been marked as a duplicate of bug 34484 ***
Comment 5 Florian Best univentionstaff 2014-07-22 09:54:20 CEST
What about the following widgets?
umc.widgets.Button
umc.widgets.SubmitButton
umc.widgets.TitlePane
umc.widgets.ExpandingTitlePane
(
umc.widgets.ContainerWidget
umc.widgets.LabelPane
)
Comment 6 Alexander Kläser univentionstaff 2014-07-22 10:01:24 CEST
(In reply to Florian Best from comment #5)
> What about the following widgets?
> umc.widgets.Button
> umc.widgets.SubmitButton
> umc.widgets.TitlePane
> umc.widgets.ExpandingTitlePane
> (
> umc.widgets.ContainerWidget
> umc.widgets.LabelPane
> )

This was not part of this bug.
Comment 7 Florian Best univentionstaff 2014-07-23 12:03:44 CEST
OK, Text.isValid() works. The other widgets not yet :(.
Comment 8 Stefan Gohmann univentionstaff 2014-11-11 15:01:43 CET
UCS 3.2-2 has been released:
 http://docs.univention.de/release-notes-3.2-2-en.html
 http://docs.univention.de/release-notes-3.2-2-de.html

If this error occurs again, please use "Clone This Bug".