[eXist-commits] [eXist-db/exist] 792688: [fix] handle IllegalStateException in file:directo...

[Adam R. <no...@gi...>]

  Branch: refs/heads/develop-6.x.x
  Home:   https://github.com/eXist-db/exist
  Commit: 7926887546769e17a6b309b77d11832305671bd1
      https://github.com/eXist-db/exist/commit/7926887546769e17a6b309b77d11832305671bd1
  Author: Juri Leino <gi...@li...>
  Date:   2023-10-16 (Mon, 16 Oct 2023)

  Changed paths:
    M extensions/modules/file/src/main/java/org/exist/xquery/modules/file/DirectoryList.java
    A extensions/modules/file/src/main/java/org/exist/xquery/modules/file/FileErrorCode.java
    A extensions/modules/file/src/test/xquery/modules/file/directory-list.xqm

  Log Message:
  -----------
  [fix] handle IllegalStateException in file:directory-list#2

fixes #5028

The Java code of DirectoryList can throw an IllegalStateException
when a non-existent directory is supplied as its first argument.
This exception class is now handled in file:list#2.

Additionally, the raised error now has a proper code `file:DIRECTORY_NOT_FOUND`.

This will allow us in the future to add proper error codes to more
errors raised by this module.


  Commit: 4d378db3961923245502768790149656b0740098
      https://github.com/eXist-db/exist/commit/4d378db3961923245502768790149656b0740098
  Author: Juri Leino <gi...@li...>
  Date:   2023-10-17 (Tue, 17 Oct 2023)

  Changed paths:
    M extensions/modules/file/src/main/java/org/exist/xquery/modules/file/FileErrorCode.java

  Log Message:
  -----------
  [refactor] address review in FileErrorCode


  Commit: 7c0f8f8c018f7a50fa306cad7428b69cf987cef8
      https://github.com/eXist-db/exist/commit/7c0f8f8c018f7a50fa306cad7428b69cf987cef8
  Author: Juri Leino <gi...@li...>
  Date:   2023-10-17 (Tue, 17 Oct 2023)

  Changed paths:
    M extensions/modules/file/src/main/java/org/exist/xquery/modules/file/DirectoryList.java

  Log Message:
  -----------
  [refactor] optimize DirectoryList for maintainability


  Commit: 8ad2fd50f70faf52bcacb427c765abfd4ac536ed
      https://github.com/eXist-db/exist/commit/8ad2fd50f70faf52bcacb427c765abfd4ac536ed
  Author: Juri Leino <gi...@li...>
  Date:   2023-10-18 (Wed, 18 Oct 2023)

  Changed paths:
    M extensions/modules/file/src/main/java/org/exist/xquery/modules/file/DirectoryList.java

  Log Message:
  -----------
  [ignore] fix formatting and modifier order


  Commit: a1ba17fda3f905acd54ca0bdc198fca4e15be079
      https://github.com/eXist-db/exist/commit/a1ba17fda3f905acd54ca0bdc198fca4e15be079
  Author: Adam Retter <ada...@go...>
  Date:   2023-10-19 (Thu, 19 Oct 2023)

  Changed paths:
    M extensions/modules/file/src/main/java/org/exist/xquery/modules/file/DirectoryList.java
    A extensions/modules/file/src/main/java/org/exist/xquery/modules/file/FileErrorCode.java
    A extensions/modules/file/src/test/xquery/modules/file/directory-list.xqm

  Log Message:
  -----------
  Merge pull request #5093 from line-o/backport/5028

[6.x.x] Handle IllegalStateException in file:directory-list#2


Compare: https://github.com/eXist-db/exist/compare/ec00524df429...a1ba17fda3f9