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

MultiCraft: Dedicated ip row indexes broken

        Details

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

          Description

          Under Multicraft -> Manage there are rows for dedicated ips. The indexing is totally broken.
          ````
          <td><?php $this->Form->fieldText('ips[][ip]', $this->Html->ifSet($vars->ips[$i]['ip']));?></td>
          <td><?php $this->Form->fieldText('ips[][port]',$this->Html->ifSet($vars->ips[$i]['port']),['class'=>'stretch port']);?></td>
          ````
          It should be
          ````
          <td><?php $this->Form->fieldText('ips[ip][]', $this->Html->ifSet($vars->ips[$i]['ip']));?></td>
          <td><?php $this->Form->fieldText('ips[port][]',$this->Html->ifSet($vars->ips[$i]['port']),['class'=>'stretch port']);?></td>
          ````
          and be converted on the backend using $this->ArrayHelper->keyToNumeric() on the ips array

            Issue Links

            is blocked by

            Bug - A problem which impairs or prevents the functions of the product. CORE-2264 Multicraft: Illegal string offset 'ip' when editing module row

            • Major - Major loss of function.
            • Closed

              Activity

              Ascending order - Click to sort in descending order
              jonathan Jonathan Reissmueller created issue -
              Automated transition triggered when Jonathan Reissmueller created a branch in Stash -
              Field Original Value New Value
              Status Open [ 1 ] In Progress [ 3 ]
              jonathan Jonathan Reissmueller made changes -
              Description Under Multicraft -> Manage there are rows for dedicated ips. The indexing is totally broken.
              ````
              <td><?php $this->Form->fieldText('*ips[][ip]*', $this->Html->ifSet($vars->ips[$i]['ip']));?></td>
              <td><?php $this->Form->fieldText('*ips[][port]*',$this->Html->ifSet($vars->ips[$i]['port']),['class'=>'stretch port']);?></td>
              ````
              It should be
              ````
              <td><?php $this->Form->fieldText('*ips[ip][]*', $this->Html->ifSet($vars->ips[$i]['ip']));?></td>
              <td><?php $this->Form->fieldText('*ips[port][]*',$this->Html->ifSet($vars->ips[$i]['port']),['class'=>'stretch port']);?></td>
              ````
              and be converted on the backend using $this->ArrayHelper->numericToKey() on both ips[ip][] and ips[ports][]               		Under Multicraft -> Manage there are rows for dedicated ips. The indexing is totally broken.
              ````
              <td><?php $this->Form->fieldText('*ips[][ip]*', $this->Html->ifSet($vars->ips[$i]['ip']));?></td>
              <td><?php $this->Form->fieldText('*ips[][port]*',$this->Html->ifSet($vars->ips[$i]['port']),['class'=>'stretch port']);?></td>
              ````
              It should be
              ````
              <td><?php $this->Form->fieldText('*ips[ip][]*', $this->Html->ifSet($vars->ips[$i]['ip']));?></td>
              <td><?php $this->Form->fieldText('*ips[port][]*',$this->Html->ifSet($vars->ips[$i]['port']),['class'=>'stretch port']);?></td>
              ````
              and be converted on the backend using $this->ArrayHelper->keyToNumeric() on both ips[ip][] and ips[ports][]
              jonathan Jonathan Reissmueller made changes -
              Description Under Multicraft -> Manage there are rows for dedicated ips. The indexing is totally broken.
              ````
              <td><?php $this->Form->fieldText('*ips[][ip]*', $this->Html->ifSet($vars->ips[$i]['ip']));?></td>
              <td><?php $this->Form->fieldText('*ips[][port]*',$this->Html->ifSet($vars->ips[$i]['port']),['class'=>'stretch port']);?></td>
              ````
              It should be
              ````
              <td><?php $this->Form->fieldText('*ips[ip][]*', $this->Html->ifSet($vars->ips[$i]['ip']));?></td>
              <td><?php $this->Form->fieldText('*ips[port][]*',$this->Html->ifSet($vars->ips[$i]['port']),['class'=>'stretch port']);?></td>
              ````
              and be converted on the backend using $this->ArrayHelper->keyToNumeric() on both ips[ip][] and ips[ports][]               		Under Multicraft -> Manage there are rows for dedicated ips. The indexing is totally broken.
              ````
              <td><?php $this->Form->fieldText('*ips[][ip]*', $this->Html->ifSet($vars->ips[$i]['ip']));?></td>
              <td><?php $this->Form->fieldText('*ips[][port]*',$this->Html->ifSet($vars->ips[$i]['port']),['class'=>'stretch port']);?></td>
              ````
              It should be
              ````
              <td><?php $this->Form->fieldText('*ips[ip][]*', $this->Html->ifSet($vars->ips[$i]['ip']));?></td>
              <td><?php $this->Form->fieldText('*ips[port][]*',$this->Html->ifSet($vars->ips[$i]['port']),['class'=>'stretch port']);?></td>
              ````
              and be converted on the backend using $this->ArrayHelper->keyToNumeric() on the ips array
              Automated transition triggered when Jonathan Reissmueller created pull request #7 in Stash -
              Status In Progress [ 3 ] In Review [ 5 ]
              Resolution Fixed [ 1 ]
              jonathan Jonathan Reissmueller made changes -
              Link This issue relates to CORE-2274 [ CORE-2274 ]
              tyson Tyson Phillips (Inactive) made changes -
              Sprint 4.0.0 Sprint 9 [ 27 ]
              tyson Tyson Phillips (Inactive) made changes -
              Rank Ranked higher
              jonathan Jonathan Reissmueller made changes -
              Link This issue blocks CORE-2264 [ CORE-2264 ]
              tyson Tyson Phillips (Inactive) made changes -
              Sprint 4.0.0 Sprint 9 [ 27 ] 4.0.0 Sprint 9, 4.0.0 Sprint 10 [ 27, 28 ]
              tyson Tyson Phillips (Inactive) made changes -
              Link This issue blocks CORE-2264 [ CORE-2264 ]
              tyson Tyson Phillips (Inactive) made changes -
              Link This issue is blocked by CORE-2264 [ CORE-2264 ]
              Automated transition triggered when Tyson Phillips (Inactive) merged pull request #7 in Stash -
              Status In Review [ 5 ] Closed [ 6 ]
              admin Paul Phillips made changes -
              Resolution Fixed [ 1 ]
              Status Closed [ 6 ] Reopened [ 4 ]
              admin Paul Phillips made changes -
              Security Private [ 10000 ]
              admin Paul Phillips made changes -
              Status Reopened [ 4 ] Closed [ 6 ]
              Resolution Fixed [ 1 ]

                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:
                    20/Oct/16

                    Agile