Uploaded image for project: 'eZ Publish / Platform'
  1. eZ Publish / Platform
  2. EZP-24613

As an editor, I want to be able to edit the ISBN fields

    Details

      Description

      It's basically similar to a Text Line except that the validation is a bit more complicated.

        Activity

        Show
        Stéphane Diot added a comment - PR: https://github.com/ezsystems/PlatformUIBundle/pull/290
        Show
        Stéphane Diot added a comment - merged in : https://github.com/ezsystems/PlatformUIBundle/commit/6757d97b0f8c64fdb3a38ceb0cc036e0953dd60e
        Hide
        Miguel das Neves Jacinto (Inactive) added a comment -

        @Stéphane Diot I am not getting the same results as shown in the screencast the only error message is "This is not a correct ISBN13 pattern" and not for example "Your ISBN13 doesn't have the correct length". Also the ISBN10 can only be tested when this issue EZP-24617 is fixed?

        Show
        Miguel das Neves Jacinto (Inactive) added a comment - @ Stéphane Diot I am not getting the same results as shown in the screencast the only error message is "This is not a correct ISBN13 pattern" and not for example "Your ISBN13 doesn't have the correct length". Also the ISBN10 can only be tested when this issue EZP-24617 is fixed?
        Hide
        Stéphane Diot added a comment -

        My bad, the screencasts were done before the final review, after that we finally changed our minds to simplify the code, and also get less error messages. That's why you don't get the same as in the pull request's screencast ( that I deleted now ).
        About ISBN10, well yes you can't test it before https://jira.ez.no/browse/EZP-24617 is fixed.

        Show
        Stéphane Diot added a comment - My bad, the screencasts were done before the final review, after that we finally changed our minds to simplify the code, and also get less error messages. That's why you don't get the same as in the pull request's screencast ( that I deleted now ). About ISBN10, well yes you can't test it before https://jira.ez.no/browse/EZP-24617 is fixed.
        Hide
        Miguel das Neves Jacinto (Inactive) added a comment -

        QA Approved

        Show
        Miguel das Neves Jacinto (Inactive) added a comment - QA Approved

          People

          • Assignee:
            Unassigned
            Reporter:
            Damien Pobel (Inactive)
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: