moqui / moqui-framework

Use Moqui Framework to build enterprise applications based on Java. It includes tools for databases (relational, graph, document), local and web services, web and other UI with screens and forms, security, file/resource access, scripts, templates, l10n, caching, logging, search, rules, workflow, multi-instance, and integration.
http://www.moqui.org
Other
284 stars 204 forks source link

Improve GeoTypeAndAssocFromDetail view entity #303

Closed gagaboy closed 6 years ago

gagaboy commented 6 years ago

In the view entity moqui.basic.GeoTypeAndAssocFromDetail,the toGeoId field is the same as geoId field .There need the fromGeoId field to relate the from associated geoId.So in this view entity definition these code `

    <alias-all entity-alias="GEOFR" prefix="from"><exclude field="geoId"/></alias-all>
</view-entity>

should be

    <alias-all entity-alias="GEOFR" prefix="from"/>
</view-entity>

`

jonesde commented 6 years ago

I'm fine with adding the fromGeoId alias, but don't like the idea of removing the existing toGeoId alias (even if it is the same via join condition as the geoId field, no point in breaking backwards compatibility).

This is done in commit 8d828a43.

For future reference: please don't create issues on GitHub. They are not yet disabled to avoid losing older issues, but all requests should now go in HiveMind on moqui.org. Please see:

https://www.moqui.org/m/docs/moqui/Community+Guide