ICEfaces
  1. ICEfaces
  2. ICE-10914

ace:fileEntry - Support client-side validation of file types

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.3
    • Component/s: None
    • Labels:
      None
    • Environment:
      Any

      Description

      As mentioned in ICE-10777, it would be worth investigating if the HTML5 file input element can support client-side validation of file-type/extension. If so, we should add support for it (on supported browsers) as this would be more efficient than the current approach of uploading the file and then performing the type validation afterwards.

        Activity

        Arturo Zambrano created issue -
        Arturo Zambrano made changes -
        Field Original Value New Value
        Assignee Arturo Zambrano [ artzambrano ]
        Ken Fyten made changes -
        Fix Version/s 4.3 [ 13096 ]
        Hide
        Arturo Zambrano added a comment -

        r52148: added client-side file type validation via 'acceptType' attribute to single upload mode

        Note that this feature doesn't depend on HTML5, so it works on most browsers.

        Show
        Arturo Zambrano added a comment - r52148: added client-side file type validation via 'acceptType' attribute to single upload mode Note that this feature doesn't depend on HTML5, so it works on most browsers.
        Ken Fyten made changes -
        Summary Investigate HTML5 file input validation capabilities ace:fileEntry - Support client-side validation of file types
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #52148 Tue Dec 12 15:54:37 MST 2017 art.zambrano ICE-10914 added client-side file type validation via 'acceptType' attribute to single upload mode
        Files Changed
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/fileentry/FileEntryRenderer.java
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/fileentry/FileEntryMeta.java
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/fileentry/fileEntry.js
        Hide
        Arturo Zambrano added a comment -

        r52167: added client-side file type validation via 'acceptType' attribute to multiple upload mode; added support to specify a general MIME type category; added dialog to display localised error message

        Show
        Arturo Zambrano added a comment - r52167: added client-side file type validation via 'acceptType' attribute to multiple upload mode; added support to specify a general MIME type category; added dialog to display localised error message
        Arturo Zambrano made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #52167 Thu Dec 14 11:07:26 MST 2017 art.zambrano ICE-10914 added client-side file type validation via 'acceptType' attribute to multiple upload mode; added support to specify a general MIME type category; added dialog to display localised error message
        Files Changed
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/resources/messages.properties
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/fileentry/FileEntryRenderer.java
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/fileentry/fileEntry.js
        Hide
        Ken Fyten added a comment -

        Can you add this to the showcase fileEntry Validation demo? Currently validates on size but this would add optional type support as well.

        Show
        Ken Fyten added a comment - Can you add this to the showcase fileEntry Validation demo? Currently validates on size but this would add optional type support as well.
        Ken Fyten made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #52421 Tue May 01 15:39:15 MDT 2018 art.zambrano ICE-10914 added Validate Type demo for ace:fileEntry
        Files Changed
        Commit graph MODIFY /icefaces4/trunk/icefaces/samples/showcase/showcase/src/main/java/org/icefaces/samples/showcase/view/menu/data/CentralDataList.java
        Commit graph ADD /icefaces4/trunk/icefaces/samples/showcase/showcase/src/main/webapp/resources/examples/ace/fileentry/fileEntryValidateType.xhtml
        Commit graph MODIFY /icefaces4/trunk/icefaces/samples/showcase/showcase/src/main/resources/org/icefaces/samples/showcase/view/resources/messages.properties
        Commit graph ADD /icefaces4/trunk/icefaces/samples/showcase/showcase/src/main/java/org/icefaces/samples/showcase/example/ace/fileentry/FileEntryValidateTypeBean.java
        Hide
        Arturo Zambrano added a comment -

        r52421: added Validate Type demo for ace:fileEntry

        Show
        Arturo Zambrano added a comment - r52421: added Validate Type demo for ace:fileEntry
        Arturo Zambrano made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #52423 Wed May 02 17:17:03 MDT 2018 art.zambrano ICE-10914 slight text modifications
        Files Changed
        Commit graph MODIFY /icefaces4/trunk/icefaces/samples/showcase/showcase/src/main/webapp/resources/examples/ace/fileentry/fileEntryValidateType.xhtml
        Commit graph MODIFY /icefaces4/trunk/icefaces/samples/showcase/showcase/src/main/resources/org/icefaces/samples/showcase/view/resources/messages.properties
        Hide
        Ken Fyten added a comment -

        Let's change the title of the new demo to "Client Validate Type" to distinguish it from the existing server-side validation, which also mentions type in the description.

        Show
        Ken Fyten added a comment - Let's change the title of the new demo to "Client Validate Type" to distinguish it from the existing server-side validation, which also mentions type in the description.
        Ken Fyten made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #52450 Tue May 15 10:16:54 MDT 2018 art.zambrano ICE-10914 changed demo name to Client Validate Type
        Files Changed
        Commit graph MODIFY /icefaces4/trunk/icefaces/samples/showcase/showcase/src/main/resources/org/icefaces/samples/showcase/view/resources/messages.properties
        Hide
        Arturo Zambrano added a comment -

        r52450: changed demo name to Client Validate Type

        Show
        Arturo Zambrano added a comment - r52450: changed demo name to Client Validate Type
        Arturo Zambrano made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #52454 Wed May 16 15:21:38 MDT 2018 art.zambrano ICE-10914 added CSS class name for the invalid type error dialog and moved default styling to this class, in order to allow developers to customize this dialog
        Files Changed
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/fileentry/fileEntry.css
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/fileentry/fileEntry.js
        Hide
        Arturo Zambrano added a comment - - edited

        r52454: added CSS class name for the invalid type error dialog and moved default styling to this class, in order to allow developers to customize this dialog

        Added this information to the wiki page.

        Show
        Arturo Zambrano added a comment - - edited r52454: added CSS class name for the invalid type error dialog and moved default styling to this class, in order to allow developers to customize this dialog Added this information to the wiki page.
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #52455 Wed May 16 15:38:17 MDT 2018 art.zambrano ICE-10914 moved padding from style attribute to CSS rule
        Files Changed
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/fileentry/fileEntry.css
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/resources/icefaces.ace/fileentry/fileEntry.js
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Arturo Zambrano
            Reporter:
            Arturo Zambrano
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: