Uploaded image for project: 'Blesta Core'
  1. Blesta Core
  2. CORE-4360

Confirmation modals don't work from public controllers

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 5.1.0-b1
    • Fix Version/s: 5.1.0
    • Component/s: None
    • Labels:
      None

      Description

      When generating a confirmation modal, we call either the admin_dialog controller or the client_dialog controller. However, there is no dialog controller that can be call via the public URI. This is a problem because we recently moved plugin nav item URLs from the client URI to the public URI which may break some existing confirmation modals.

      We should add a public dialog controller.

      To reproduce the broken dialog edit plugins/support_manager/views/default/client_tickets.pdt and change

                  base_url: '<?php echo (isset($this->client_uri) ? $this->Html->safe($this->client_uri) : null);?>',
      

      To

                  base_url: '<?php echo (isset($this->base_uri) ? $this->Html->safe($this->base_uri) : null);?>',
      

      Then login as a client, visit /plugin/support_manager/client_tickets/, and try to close a ticket.

        Activity

        jonathan Jonathan Reissmueller created issue -
        jonathan Jonathan Reissmueller made changes -
        Field Original Value New Value
        Rank Ranked higher
        jonathan Jonathan Reissmueller made changes -
        Sprint 5.2.0 Sprint 1 [ 137 ]
        jonathan Jonathan Reissmueller made changes -
        Rank Ranked lower
        jonathan Jonathan Reissmueller made changes -
        Rank Ranked higher
        jonathan Jonathan Reissmueller made changes -
        Assignee Jonathan Reissmueller [ jonathan ]
        jonathan Jonathan Reissmueller made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        jonathan Jonathan Reissmueller made changes -
        Description When generating a confirmation modal, we call either the admin_dialog controller or the client_dialog controller. However, there is no dialog controller that can be call via the public URI. This is a problem because we recently moved plugin nav item URLs from the client URI to the public URI which may break some existing confirmation modals.

        We should add a public dialog controller.
        When generating a confirmation modal, we call either the admin_dialog controller or the client_dialog controller. However, there is no dialog controller that can be call via the public URI. This is a problem because we recently moved plugin nav item URLs from the client URI to the public URI which may break some existing confirmation modals.

        We should add a public dialog controller.

        To reproduce the broken dialog edit plugins/support_manager/views/default/client_tickets.pdt and change

        {code:java}
                    base_url: '<?php echo (isset($this->client_uri) ? $this->Html->safe($this->client_uri) : null);?>',
        {code}

        To

        {code:java}
                    base_url: '<?php echo (isset($this->base_uri) ? $this->Html->safe($this->base_uri) : null);?>',
        {code}

        Then login as a client, visit /plugin/support_manager/client_tickets/, and try to cancel a ticket.
        jonathan Jonathan Reissmueller made changes -
        Description When generating a confirmation modal, we call either the admin_dialog controller or the client_dialog controller. However, there is no dialog controller that can be call via the public URI. This is a problem because we recently moved plugin nav item URLs from the client URI to the public URI which may break some existing confirmation modals.

        We should add a public dialog controller.

        To reproduce the broken dialog edit plugins/support_manager/views/default/client_tickets.pdt and change

        {code:java}
                    base_url: '<?php echo (isset($this->client_uri) ? $this->Html->safe($this->client_uri) : null);?>',
        {code}

        To

        {code:java}
                    base_url: '<?php echo (isset($this->base_uri) ? $this->Html->safe($this->base_uri) : null);?>',
        {code}

        Then login as a client, visit /plugin/support_manager/client_tickets/, and try to cancel a ticket.
        When generating a confirmation modal, we call either the admin_dialog controller or the client_dialog controller. However, there is no dialog controller that can be call via the public URI. This is a problem because we recently moved plugin nav item URLs from the client URI to the public URI which may break some existing confirmation modals.

        We should add a public dialog controller.

        To reproduce the broken dialog edit plugins/support_manager/views/default/client_tickets.pdt and change

        {code:java}
                    base_url: '<?php echo (isset($this->client_uri) ? $this->Html->safe($this->client_uri) : null);?>',
        {code}

        To

        {code:java}
                    base_url: '<?php echo (isset($this->base_uri) ? $this->Html->safe($this->base_uri) : null);?>',
        {code}

        Then login as a client, visit /plugin/support_manager/client_tickets/, and try to close a ticket.
        jonathan Jonathan Reissmueller made changes -
        Remaining Estimate 0 minutes [ 0 ]
        Time Spent 20 minutes [ 1200 ]
        Worklog Id 15079 [ 15079 ]
        jonathan Jonathan Reissmueller made changes -
        Status In Progress [ 3 ] In Review [ 5 ]
        Resolution Fixed [ 1 ]
        abdy Abdy Franco made changes -
        Time Spent 20 minutes [ 1200 ] 32 minutes [ 1920 ]
        Worklog Id 15082 [ 15082 ]
        jonathan Jonathan Reissmueller made changes -
        Status In Review [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            jonathan Jonathan Reissmueller
            Reporter:
            jonathan Jonathan Reissmueller
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:
              Fix Release Date:
              6/Jul/21

              Time Tracking

              Estimated:
              Original Estimate - Not Specified
              Not Specified
              Remaining:
              Remaining Estimate - 0 minutes
              0m
              Logged:
              Time Spent - 32 minutes
              32m

                Agile