[MIG] base_geoengine: Migrated to 19.0 by weinni2000 · Pull Request #414 · OCA/geospatial

weinni2000 · 2025-10-04T16:25:07Z

Major points:

moved the _condition_to_sql to the change classes, dealt with the changed function signatures etc
created the new optimization functions for geospatial fields which are necessary in odoo19
Standardize translation calls to use env._ and minor odoo 19 syntax

…d create multipolygon

…es and add a menu

… geoengine view

…view in geo_edit_map

Before this change, simple geometry column were created without inforamtion regarding the geo type nor the defined SRID. Without these information, the srid has to be expliciltely set when converting data to column format.

…had to be use. Add the possibility to have a right internal domain on the right

…method with our alias to perform search method and fix tests

Currently translated at 77.9% (127 of 163 strings) Translation: geospatial-16.0/geospatial-16.0-base_geoengine Translate-URL: https://translation.odoo-community.org/projects/geospatial-16-0/geospatial-16-0-base_geoengine/sv/

Currently translated at 100.0% (163 of 163 strings) Translation: geospatial-16.0/geospatial-16.0-base_geoengine Translate-URL: https://translation.odoo-community.org/projects/geospatial-16-0/geospatial-16-0-base_geoengine/it/

@reichie020212

> > Co-authored-by: Antoni Marroig Campomar <amarroig@apsl.net> Co-authored-by: Red Butay <@reichie020212> Co-authored-by: Miquel Alzanillas <malzanillas@apsl.net> Co-authored-by: Sergio Sancho <sersanchus@gmail.com> "

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: geospatial-17.0/geospatial-17.0-base_geoengine Translate-URL: https://translation.odoo-community.org/projects/geospatial-17-0/geospatial-17-0-base_geoengine/

Currently translated at 100.0% (168 of 168 strings) Translation: geospatial-17.0/geospatial-17.0-base_geoengine Translate-URL: https://translation.odoo-community.org/projects/geospatial-17-0/geospatial-17-0-base_geoengine/it/

Currently translated at 100.0% (168 of 168 strings) Translation: geospatial-17.0/geospatial-17.0-base_geoengine Translate-URL: https://translation.odoo-community.org/projects/geospatial-17-0/geospatial-17-0-base_geoengine/zh_CN/

weinni2000 · 2025-10-04T16:37:13Z

Here are the same changes against an integration branch to see the changes:
weinni2000#4

weinni2000 · 2025-10-29T17:16:25Z

I refactored it to not use odoo-test-helpers anymore (which was not working in odoo19).
See discussion: OCA/odoo-test-helper#34
Now everything is green

juppe · 2026-02-23T13:32:20Z

Any updates on when this PR might be merged?

weinni2000 · 2026-02-23T14:41:58Z

It needs 2 reviews to be dealt with by the maintainers.

There is still an issue in the code which is not covered from the test. But I was hoping somebody else smarter than me is fixing it ;). So you are welcome to review fix it.

yvaucher · 2026-02-25T09:15:54Z

Hello @weinni2000 thanks for the work done here.

Can you add the following commits that just were FW from version 16.0 to 17.0 and 18.0 ?

#431

juppe · 2026-03-06T11:08:31Z

It needs 2 reviews to be dealt with by the maintainers.

There is still an issue in the code which is not covered from the test. But I was hoping somebody else smarter than me is fixing it ;). So you are welcome to review fix it.

@weinni2000 I have been testing your branch in version 19.0 for a few weeks. I made a pull request to your fork fixing some errors that I have run into. In addition to fixing the errors, I have also updated the OpenLayers, Chroma.js, Geostats JS libraries.

weinni2000#7

ps. I also cherry-picked in the commits from #431

weinni2000 and others added 30 commits October 4, 2025 11:54

[IMP] geoengine_base: pre-commit auto fixes

16d3789

[UPD] README.rst

91e77b8

[IMP] base_geoengine: black, isort, prettier

052221d

[MIG] base_geoengine : migration from odoo 14 to odoo 16

d6056dc

[ADD] base_geoengine: display field geo_edit_map with multipolygon an…

1c6fe8d

…d create multipolygon

[FIX] base_geoengine: fix zip creation and refractor geo_edit_map

d3fa114

[FIX] base_geoengine: fix retails machine and start geoengine view fix

ba35530

[FIX] base_geoengine: fix geoengine view renderer

7d5a986

[ADD] base_geoengine: add left-side menu for layers

b35ef10

[FIX] base_geoengine: display raster layers for zip and retail machin…

db03436

…es and add a menu

[FIX] base_geoengine: display vector layers for zip and retail machines

b31db8b

[FIX] remove ref to submodule

faf596e

[FIX] base_geoengine: display info box

d9a0064

[FIX] base_geoengine: button to edit record on info box + fix display…

9089d14

… geoengine view

[FIX] base_geogines: fix bugs and use opacity of the layer management

065cf20

[FIX] base_geoengine: code cleaning

233933e

[FIX] base_geoengine: fix pre-commit

ec21895

[IMP][FIX] base_geoengine: start model tests and fix retail machines …

4804091

…view in geo_edit_map

[IMP] base_geoengine: end model tests

b550b42

changing the default SRID (correction of the previous one)

42221bd

[IMP] base_geoengine: add tests from_lat_lonand to_lat_lon

88a59f4

[IMP] base_geoengine: refactor geo_search

638b3bd

[FIX] base_geoengine: current_field __leaf_to_sql

6d6515a

[FIX] base_geoengine: Display Labels on Polygon

58ffa2c

[FIX] base_geoengine: fix srid

539ec39

[FIX] base_geoengine: fix column definition at install

ce6476d

Before this change, simple geometry column were created without inforamtion regarding the geo type nor the defined SRID. Without these information, the srid has to be expliciltely set when converting data to column format.

[IMP] base_geoengine: geo_search is deprecated, normal search method …

3f11eaf

…had to be use. Add the possibility to have a right internal domain on the right

[FIX] base_geoengine: fix __leaf_to_sql by adding our own where_calc …

dcea2fb

…method with our alias to perform search method and fix tests

[FIX] base_geoengine: fix fields SRID

e6ab1cd

[IMP] base_geoengine : Access rules

9a988f7

jakobkrabbe and others added 19 commits October 4, 2025 18:02

Translated using Weblate (Swedish)

72bc031

Currently translated at 77.9% (127 of 163 strings) Translation: geospatial-16.0/geospatial-16.0-base_geoengine Translate-URL: https://translation.odoo-community.org/projects/geospatial-16-0/geospatial-16-0-base_geoengine/sv/

Added translation using Weblate (Italian)

8fdcc2e

Translated using Weblate (Italian)

c6f88da

Currently translated at 100.0% (163 of 163 strings) Translation: geospatial-16.0/geospatial-16.0-base_geoengine Translate-URL: https://translation.odoo-community.org/projects/geospatial-16-0/geospatial-16-0-base_geoengine/it/

[IMP] base_geoengine: pre-commit auto fixes

65ae774

[MIG] base_geoengine: Migration to 17.0

ff0bf48

> > Co-authored-by: Antoni Marroig Campomar <amarroig@apsl.net> Co-authored-by: Red Butay <@reichie020212> Co-authored-by: Miquel Alzanillas <malzanillas@apsl.net> Co-authored-by: Sergio Sancho <sersanchus@gmail.com> "

[UPD] Update base_geoengine.pot

05c2204

[BOT] post-merge updates

f9dd896

Update translation files

0f3c248

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: geospatial-17.0/geospatial-17.0-base_geoengine Translate-URL: https://translation.odoo-community.org/projects/geospatial-17-0/geospatial-17-0-base_geoengine/

Translated using Weblate (Italian)

c90294f

Currently translated at 100.0% (168 of 168 strings) Translation: geospatial-17.0/geospatial-17.0-base_geoengine Translate-URL: https://translation.odoo-community.org/projects/geospatial-17-0/geospatial-17-0-base_geoengine/it/

[UPDATE] Add rtl ignore for persian and arabic language

0a8d11d

Added translation using Weblate (Chinese (Simplified) (zh_CN))

76fdbf1

[BOT] post-merge updates

fcd6052

Translated using Weblate (Chinese (Simplified) (zh_CN))

e9ec494

Currently translated at 100.0% (168 of 168 strings) Translation: geospatial-17.0/geospatial-17.0-base_geoengine Translate-URL: https://translation.odoo-community.org/projects/geospatial-17-0/geospatial-17-0-base_geoengine/zh_CN/

[IMP] base_geoengine: pre-commit auto fixes

5261c31

[18.0][MIG] base_geoengine: Migration to 18.0

734f962

[18.0][UPD] upgrade version of libs used in render

9b9bfc3

[UPD] Update base_geoengine.pot

54dd285

[BOT] post-merge updates

654f697

[IMP] base_geoengine: pre-commit auto fixes

a460a1b

weinni2000 changed the title ~~19.0 mig base geoengine~~ [MIG] base_geoengine: Migrated to 17.0 Oct 4, 2025

weinni2000 changed the title ~~[MIG] base_geoengine: Migrated to 17.0~~ [MIG] base_geoengine: Migrated to 19.0 Oct 4, 2025

[MIG] base_geoengine: Migration to 19.0

07b65c6

weinni2000 force-pushed the 19.0-mig-base_geoengine branch from 5c4fead to 07b65c6 Compare October 29, 2025 17:12

Merge branch '19.0' into 19.0-mig-base_geoengine

1614afd

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

[MIG] base_geoengine: Migrated to 19.0#414

[MIG] base_geoengine: Migrated to 19.0#414
weinni2000 wants to merge 99 commits intoOCA:19.0from
weinni2000:19.0-mig-base_geoengine

weinni2000 commented Oct 4, 2025

Uh oh!

weinni2000 commented Oct 4, 2025

Uh oh!

weinni2000 commented Oct 29, 2025

Uh oh!

juppe commented Feb 23, 2026

Uh oh!

weinni2000 commented Feb 23, 2026

Uh oh!

yvaucher commented Feb 25, 2026

Uh oh!

juppe commented Mar 6, 2026 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

14 participants

Uh oh!

Conversation

weinni2000 commented Oct 4, 2025

Uh oh!

weinni2000 commented Oct 4, 2025

Uh oh!

weinni2000 commented Oct 29, 2025

Uh oh!

juppe commented Feb 23, 2026

Uh oh!

weinni2000 commented Feb 23, 2026

Uh oh!

yvaucher commented Feb 25, 2026

Uh oh!

juppe commented Mar 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

14 participants

juppe commented Mar 6, 2026 •

edited

Loading