magloire / vidisearch

2 stars 2 forks source link

Submoduler og registrering #4

Open baffioso opened 6 years ago

baffioso commented 6 years ago

@magloire @mapcentia Har lig lagt Martins betragtninger her. Kunne være fint, hvis i køre kommunikationen herinde, så jeres beslutninger bliver dokumenteret åbent. Jeg har cloned projektet ud og installeret det lokalt. Nu ved jeg ikke helt hvad der er tænkt med sub-modulers placering (som alm. moduler?), men jeg tænker, at det kunne være godt at have dem som sub-git-projekter i vidisearch modulet, så de er samlet og kan enables samlet:

vidisearch
              |_____ index.js
              |_____ dawaSearcher
              |                            |_____ index.js
              |_____ matrikelSearcher
              |                            |_____ index.js
              |_____ planSearcher
              |                            |_____ index.js
              |_____ et_andet_git_repo
                                           |_____ index.js

Så så kunne aktivere dem således:

browser: [
             {
                 vidisearch: [
                     "index",
                     "dawaSearcher/index",
                     "matrikelSearcher/index",
                     "planSearcher/index",
                     "et_andet_git_repo/index"
                 ]
             }
]

Det kræver et lille ændring i Vidi, som jeg lige har lagt ud: https://github.com/mapcentia/vidi/blob/develop/browser/modules/init.js#L229

Endvidere kan du i et sub-module for fat på mainSearch således i stedet for at require det:

set: function (o) {
         cloud = o.cloud;
         utils = o.utils;
         backboneEvents = o.backboneEvents;
         mapObj = cloud.get().map;
         mainSearch = o.extensions.vidisearch.index;

},

Her får du også det initierede modul og ikke et nyt.

baffioso commented 6 years ago

@magloire Prøvede lige nedenstående registrering af submoduler i vidi's config.js, men kan desværre ikke få det til at virke. Er det på den måde du har registreret submodulerne?

browser: [
             {
                 vidisearch: [
                     "index",
                     "dawaSearcher/index",
                     "matrikelSearcher/index",
                     "planSearcher/index",
                     "et_andet_git_repo/index"
                 ]
             }
]
magloire commented 6 years ago

@baffioso sådan har jeg registreret submodulerne. Det ser ud til du gør det samme.

config

husk at have submodulerne under vidisearch mappen: config1

baffioso commented 6 years ago

Takker