Opened 6 years ago

#4878 new defect

GUI should throw errors if directory includes failed, VFS::DirectoryExists

Reported by: elexis Owned by:
Priority: Nice to Have Milestone: Backlog
Component: Core engine Keywords:
Cc: Patch:

Description

If a GUI page file wants to include a JS file that doesn't exist, it throws an error:

For instance:

<script file="nope/doesntexist.js"/>

results in

ERROR: File 'nope/doesntexist.js' does not exist

But for directories it silently continues and just doesn't load any files if there was a typo in the directory name:

<script directory="doesnt/exist/either/"/>

Specifically mentioned includes should never be optional and hence always throw an error.

On first sight it looks like we are missing a DirectoryExists test that we don't have for the VFS.

refs Phab:D619

Change History (0)

Note: See TracTickets for help on using tickets.