search

digicademy / svsal

The School of Salamanca. Web Application
MIT License
16 stars 7 forks source link

Split app.xql ? #79

Closed dvglc closed 4 years ago

dvglc commented 4 years ago

The app module has become somewhat confusing in that it does provide content for many different, not necessarily related functionalities (reading view, catalogue records, working papers, author/lemma pages) at once.

I think it would be worthwile splitting this into "thematic" modules (such as catrecord.xqm, work.xqm, etc.) which include functions only for the respective functionality. This would make application design more intelligible and, perhaps, also could address performance issues.

dvglc commented 4 years ago

I'm not sure of the benefits of splitting the app.xqm would really outweigh the costs of time at this moment, so I'm closing this for now.