camptocamp / c2cgeoportal

c2cgeoportal application
http://geomapfish.org
Other
64 stars 46 forks source link

GeoMapFish 1.4.1.1 : troubles with mobile application #888

Closed gnerred closed 10 years ago

gnerred commented 10 years ago

I've installed the latest release 1.4.1.1, and now my mobile application (http://preprod.cartoriviera.ch/preprod/wsgi/mobile/) has several problems :

1°/ At first loading, all layers are displayed in address bar -> more than 15'000 characters... 2°/ At first loading, the name of the theme selected contains all those characters... 3°/ When changing theme to one without layers selected, I get bad WMS requests 4°/ Icons of themes are not displayed

Did I miss something or is there a problem with this version ?

Thanks

kalbermattenm commented 10 years ago

Uh ???

I also just installed version 1.4.1.1 (on my computer....), but I don't see any problem on my side... weird...

I'll just update one of our Internet instance and keep you up-to-date.

gnerred commented 10 years ago

Okay... In the changelog, there is :

        + App.themes = '${themes | n}';
        + App.theme = '${theme | n}';
        + App.WFSTypes = '${wfs_types | n}';

It seemed strange to me, but I applied. Now I have removed the second line, and all works weel !

So thanks @sbrunner to correct the changelog...

sbrunner commented 10 years ago

The ChangeLog is correct if you remove it the theme isn't working from the default and permalink theme ...

gnerred commented 10 years ago

Sorry but I don't really understand... could you please tell me what is not workink on http://preprod.cartoriviera.ch/preprod/wsgi/mobile/ ?

sbrunner commented 10 years ago

I suspect that the permalink don't work, it's working but I don't know how :( but If you set a default_permalink_theme it's not working, the first one will be get...

gnerred commented 10 years ago

So I have now that in my config.js :

App.raster = true; App.themes = '${themes | n}'; App.theme = '${themes | n}'; App.WFSTypes = '${wfs_types | n}'; App.info = '${info | n}';

And you can see that my mobile application is not working anymore... What is the problem ? @kalbermattenm, what is the situation on your side ?

kalbermattenm commented 10 years ago

Strange... I just tested your mobile application and I didn't see any problem...

On my side, my prepub app is stuck on the mobile app and I don't know why, some kind of strange mako rendering exception, but it is actually working on all others instances...

I'm almost sure that my problem has nothing to do with yours... but I would like to make it work on my side to check if I have the same issues as you have...

kalbermattenm commented 10 years ago

Ok... I'm back on track (don't ask what happened, I don't know...)

http://sitn.ne.ch/prepub/wsgi/mobile/

Seems everything is working... (except feature highlight, see https://github.com/camptocamp/c2cgeoportal/issues/889)

gnerred commented 10 years ago

@kalbermattenm are you sure you have tested my preprod ? With which device ? I have this problem in Chrome desktop, Samsung Tab and iPhone both on internet and with Browserstack... Maybe you have an old application in you browser cache... @sbrunner, have you an idea of the problem ?

kalbermattenm commented 10 years ago

@gnerred, you're right... my cache...

Have you defined a default theme in the config.yaml file ? I'm looking for differences between your app and sitn... (because sitn is working on 1.4.1.1)

gnerred commented 10 years ago

Yes I have that in my config.yaml.in :

functionalities: anonymous: print_template:

I made a special theme for the default mobile theme

kalbermattenm commented 10 years ago

Me too... isn't that...

gnerred commented 10 years ago

And @pgiraud , any idea of the problem ? Thanks

pgiraud commented 10 years ago

The following is wrong :

App.theme = '${themes | n}';

It should be :

App.theme = '${theme | n}';
gnerred commented 10 years ago

Aaaaargghhhhhh.... thanks @pgiraud ! My fault :O( I really have to buy glasses....

sbrunner commented 10 years ago

@gnerred I sorry don't read my notification for a time ... It seam to be OK, can we close the issue ?

gnerred commented 10 years ago

Yes you can :O)

sbrunner commented 10 years ago

Thanks :-)