Skip to content

[FW] [FIX] base_geoengine: Fix Vector Layer related Error - #476

Merged
OCA-git-bot merged 2 commits into
OCA:18.0from
PyTech-SRL:18.0-fix-geoview
Oct 6, 2026
Merged

OCA-git-bot merged 2 commits into
OCA:18.0from
PyTech-SRL:18.0-fix-geoview

Conversation

@anusriNPS

@anusriNPS anusriNPS commented Aug 14, 2026 •

Copy link
Copy Markdown

FW #417

JS Error is not observed in 18.0. Porting the changes as notifying the users is useful when attributes selected in vector layer is not defined in geoengine view templates

@anusriNPS anusriNPS changed the title [IMP] base_geoengine: Fix Vector Layer related Error [18.0] [IMP] base_geoengine: Fix Vector Layer related Error Aug 14, 2026
@anusriNPS
anusriNPS marked this pull request as draft September 14, 2026 07:32
@anusriNPS
anusriNPS force-pushed the 18.0-fix-geoview branch 2 times, most recently from 8233c79 to 33e629f Compare September 15, 2026 05:00
@anusriNPS
anusriNPS marked this pull request as ready for review September 16, 2026 03:49
@anusriNPS

Copy link
Copy Markdown
Author
image

@max3903 max3903 changed the title [18.0] [IMP] base_geoengine: Fix Vector Layer related Error [18.0] [FIX] base_geoengine: Fix Vector Layer related Error Oct 5, 2026
@max3903 max3903 added this to the 18.0 milestone Oct 5, 2026

@max3903 max3903 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the FW of #417.

Please guard missing attribute_field_id in checkAttributeFieldUsage (it’s optional):

if (!cfg.attribute_field_id) {
    return true;
}

Happy to approve after that.

@anusriNPS anusriNPS changed the title [18.0] [FIX] base_geoengine: Fix Vector Layer related Error [FW] [FIX] base_geoengine: Fix Vector Layer related Error Oct 6, 2026
    Notifying user to define selected attribute_field_id of
supported type from vector layer as part of geoengine view
xml defintion which avoids observed JS error.
   Updated checkAttributeFieldUsage method in order to
handle a vector layer which does not contain
attribute_field_id value as the field
attribute_field_id is optional for basic representation
mode in geoengine view.
@anusriNPS

Copy link
Copy Markdown
Author

Thanks for the FW of #417.

Please guard missing attribute_field_id in checkAttributeFieldUsage (it’s optional):

if (!cfg.attribute_field_id) {
    return true;
}

Happy to approve after that.

Updated as suggested. Backported same to branch 16.0

@max3903 max3903 self-assigned this Oct 6, 2026
@max3903

max3903 commented Oct 6, 2026

Copy link
Copy Markdown
Member

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

On my way to merge this fine PR!
Prepared branch 18.0-ocabot-merge-pr-476-by-max3903-bump-patch, awaiting test results.

@max3903 max3903 added the bug label Oct 6, 2026
@OCA-git-bot
OCA-git-bot merged commit e113fad into OCA:18.0 Oct 6, 2026
7 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at c93987b. Thanks a lot for contributing to OCA. ❤️

@HekkiMelody
HekkiMelody deleted the 18.0-fix-geoview branch October 8, 2026 11:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants