Page MenuHomePhorge

Fix editing Conpherence rooms on mobile
ClosedPublic

Authored by aklapper on Feb 15 2025, 14:14.
Tags
None
Referenced Files
F3838562: D25879.1746097008.diff
Wed, Apr 30, 10:56
F3675866: D25879.1745529010.diff
Wed, Apr 23, 21:10
F3675200: D25879.1745503827.diff
Wed, Apr 23, 14:10
F3673624: D25879.1745476328.diff
Wed, Apr 23, 06:32
F3648709: D25879.1745429712.diff
Tue, Apr 22, 17:35
F3620609: D25879.1745311274.diff
Mon, Apr 21, 08:41
F3604985: D25879.1745247889.diff
Sun, Apr 20, 15:04
F3604563: D25879.1745211611.diff
Sun, Apr 20, 05:00

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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Feb 17 2025, 07:39