Uploaded image for project: 'OFBiz'
  1. OFBiz
  2. OFBIZ-6148

Adding GeoPoints overview to geo management

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Implemented
    • Affects Version/s: Trunk
    • Fix Version/s: 16.11.01
    • Component/s: framework
    • Labels:
    • Sprint:
      Community Day 1 - 2015

      Description

      Currently an overview of geo points is missing in webtools/geomanagement

      1. OFBIZ-6148-GeoPointsUiLabels.patch
        0.8 kB
        Pierre Smits
      2. OFBIZ-6148-GeoPointsController.patch
        2 kB
        Pierre Smits
      3. OFBIZ-6148-GeoPoints.patch
        10 kB
        Pierre Smits

        Activity

        Hide
        pfm.smits Pierre Smits added a comment -

        Thanks Jacques.

        Show
        pfm.smits Pierre Smits added a comment - Thanks Jacques.
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Thanks Pierre,

        Your patch is in trunk at revision: 1667969

        Show
        jacques.le.roux Jacques Le Roux added a comment - Thanks Pierre, Your patch is in trunk at revision: 1667969
        Hide
        pfm.smits Pierre Smits added a comment -

        The patch delivers an overview of geo points.

        What you expected might be in a next issue/patch. Not having it in this doesn't break functionality.

        Show
        pfm.smits Pierre Smits added a comment - The patch delivers an overview of geo points. What you expected might be in a next issue/patch. Not having it in this doesn't break functionality.
        Hide
        pfm.smits Pierre Smits added a comment -

        Adding missing label.

        Show
        pfm.smits Pierre Smits added a comment - Adding missing label.
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Better but there is no WebtoolsGeoPoints label

        Also I'd have expected that the geo point would have been rendered on a Geo Map, like we have
        https://localhost:8443/partymgr/control/PartyGeoLocation?partyId=FrenchCustomer (OK that's my house )

        Show
        jacques.le.roux Jacques Le Roux added a comment - Better but there is no WebtoolsGeoPoints label Also I'd have expected that the geo point would have been rendered on a Geo Map, like we have https://localhost:8443/partymgr/control/PartyGeoLocation?partyId=FrenchCustomer (OK that's my house )
        Hide
        pfm.smits Pierre Smits added a comment -

        I accidentally forgot to include the changes in controller.xml. This patch addresses that aspect.

        Thanks for spotting it, Jacques.

        Show
        pfm.smits Pierre Smits added a comment - I accidentally forgot to include the changes in controller.xml. This patch addresses that aspect. Thanks for spotting it, Jacques.
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        You missed the ctlr: org.ofbiz.webapp.control.RequestHandlerException: Unknown request [geoPoints]; this request does not exist or cannot be called directly.

        Show
        jacques.le.roux Jacques Le Roux added a comment - You missed the ctlr: org.ofbiz.webapp.control.RequestHandlerException: Unknown request [geoPoints] ; this request does not exist or cannot be called directly.
        Hide
        pfm.smits Pierre Smits added a comment -

        This patch addresses the issue.

        Show
        pfm.smits Pierre Smits added a comment - This patch addresses the issue.

          People

          • Assignee:
            pfm.smits Pierre Smits
            Reporter:
            pfm.smits Pierre Smits
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development

                Agile