Ticket #575 (closed enhancement: fixed)

Opened 20 months ago

Last modified 17 months ago

base MapFish on Pylons 1.0

Reported by: elemoine Owned by: elemoine
Priority: blocker Milestone: framework.server 2.0 Release
Component: framework.server Version: SVN
Keywords: Cc:
State: Commit

Description

summary says it all.

Attachments

patch-server-575-A0.diff (8.3 kB) - added by elemoine 20 months ago.
patch-sample-575-A0.diff (16.1 kB) - added by elemoine 20 months ago.

Change History

Changed 20 months ago by elemoine

Changed 20 months ago by elemoine

  Changed 20 months ago by elemoine

  • state set to Review

This is ready for review.

  Changed 20 months ago by bbinet

patch-server-575-A0.diff looks good, but it is funny that you forgot to update Pylons dependency version in setup.py whereas the ticket is named: "base MapFish on Pylons 1.0" ;)

We could also remove GeoAlchemy Geometry argument "dimension=2" since it is the default and does not add much value. Same thing for SqlAlchemy sessionmaker arguments: "autoflush=True", "autocommit=False" are the defaults, see: http://www.sqlalchemy.org/docs/reference/orm/sessions.html#sqlalchemy.orm.sessionmaker

follow-up: ↓ 5   Changed 20 months ago by bbinet

  • state changed from Review to Commit

concerning patch-sample-575-A0.diff, it looks good to me, but I haven't tested mapfishsample application for real. So if it works for you, feel free to commit (after having updated setup.py dependencies with Pylons 1.0).

Just one question: I saw that mapfishsample init_model method contains specific code related to spatial lite connexion: is this code specific to mapfishsample of should it be in MapFish model template? (IMO, I would just document these database specificities somewhere in MapFish documentation).

  Changed 20 months ago by elemoine

  • status changed from new to closed
  • resolution set to fixed

(In [3580]) base Map Fish? on Pylons 1.0, r=bbinet (bbinet's review comments taken into account) (closes #575)

in reply to: ↑ 3   Changed 20 months ago by elemoine

Replying to bbinet:

concerning patch-sample-575-A0.diff, it looks good to me, but I haven't tested mapfishsample application for real. So if it works for you, feel free to commit (after having updated setup.py dependencies with Pylons 1.0). Just one question: I saw that mapfishsample init_model method contains specific code related to spatial lite connexion: is this code specific to mapfishsample of should it be in MapFish model template? (IMO, I would just document these database specificities somewhere in MapFish documentation).

Hi Bruno. Thanks for your review. This is already documented in the MapFish docs, see <http://trac.mapfish.org/trac/mapfish/browser/framework/server/trunk/docs/framework/spatial-databases.txt#L333>.

  Changed 17 months ago by fredj

(In [3613]) remove unneeded directories, model dir is created by pylons (see #575)

Note: See TracTickets for help on using tickets.