feat: smaller improvements in view module

- restructure library imports: external then internal
- standards compliant output declaration
- lookup without try-catch
- disallow legacy class-syntax
- filter data-template attributes

Author: line-o

src/main/xar-resources/modules/view.xql

@@ -1,24 +1,33 @@
 xquery version "3.1";
 
-import module namespace app="http://exist-db.org/xquery/app" at "app.xql";
-import module namespace config="http://exist-db.org/xquery/apps/config" at "config.xqm";
 
 import module namespace templates="http://exist-db.org/xquery/html-templating";
 import module namespace lib="http://exist-db.org/xquery/html-templating/lib";
 
-declare option exist:serialize "method=html5 media-type=text/html";
+(: The following modules provide functions which will be called by the templating :)
+import module namespace config="http://exist-db.org/xquery/apps/config" at "config.xqm";
+import module namespace app="http://exist-db.org/xquery/app" at "app.xql";
+
 
-let $config := map {
+declare namespace output="http://www.w3.org/2010/xslt-xquery-serialization";
+
+declare option output:method "html5";
+declare option output:media-type "text/html";
+
+declare function local:lookup ($func as xs:string, $arity as xs:integer) as function(*)? {
+    function-lookup(xs:QName($func), $arity)
+};
+
+declare variable $local:templating-configuration := map {
     $templates:CONFIG_APP_ROOT : $config:app-root,
+    $templates:CONFIG_USE_CLASS_SYNTAX : false(),
+    $templates:CONFIG_FILTER_ATTRIBUTES : true(),
     $templates:CONFIG_STOP_ON_ERROR : true()
-}
-let $resolve := function($func as xs:string, $arity as xs:integer) {
-    try {
-        function-lookup(xs:QName($func), $arity)
-    } catch * {
-        ()
-    }
-}
-let $content := request:get-data()
-return
-    templates:apply($content, $resolve, (), $config)
+};
+
+templates:apply(
+    request:get-data(),
+    local:lookup#2,
+    (),
+    $local:templating-configuration
+)