Page MenuHomePhorge

Fix editing Conpherence rooms on mobile
AcceptedPublic

Authored by aklapper on Sat, Feb 15, 14:14.
Tags
None
Referenced Files
F2982815: D25879.1739971917.diff
Tue, Feb 18, 13:31
F2982768: D25879.1739971806.diff
Tue, Feb 18, 13:30
F2979409: D25879.1739852768.diff
Mon, Feb 17, 04:26
F2979403: D25879.1739852371.diff
Mon, Feb 17, 04:19
F2979395: D25879.1739851897.diff
Mon, Feb 17, 04:11
F2979147: D25879.1739792833.diff
Sun, Feb 16, 11:47
F2979041: D25879.1739761050.diff
Sun, Feb 16, 02:57
F2979034: D25879.1739759070.diff
Sun, Feb 16, 02:24

Details

Summary

rPd6bce34a5db1a838a988440f09f7728747c9e067 failed to replace all occurrences of $this->getApplicationURI('update/'.$conpherence->getID().'/') with $this->getApplicationURI('edit/'.$conpherence->getID().'/'). Thus editing a Conpherence room on mobile crashes due to using an old invalid path.

Closes T15513

Test Plan
  • Run grep -r "update/" . | grep onpherenc vs grep -r "edit/" . | grep onpherenc and read rPd6bce34a5db1a838a988440f09f7728747c9e067
  • Go to a Conpherence room in mobile view and select "Edit Room"
  • Go to a Conpherence room in desktop view and select "Edit Room"

Diff Detail

Repository
rP Phorge
Branch
T15513 (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 1705
Build 1705: arc lint + arc unit