User tests: Successful: Unsuccessful:
Same fixes as in #4916 for these components.
For any of these items:
Change the Title/name
Press Save as Copy.
Instead of a figure, the new item will get the new title/name and new alias based on the title/name
Thanks original code is by @joomdonation
Labels |
Added:
?
|
Thanks JM. Could you please change the the line of code from:
$origTable->load((int) $input->getInt('id'));
To:
$origTable->load($input->getInt('id'));
We call $input->getInt('id'), so the data is sanitized to integer already and no need for (int) before.
Please help making the change in the article model as well.
From my quick review, It looks OK. I will test it with the mentioned components when I get home later today.
Done as well as ither code style issues.
Corrected article directly with another cs too
Rel_Number | ⇒ | 4916 | |
Relation Type | ⇒ | Related to |
@test OK
Title |
|
||||||
Status | Pending | ⇒ | Closed | ||||
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2014-10-28 07:06:22 |
Same issue for weblinks: needs a separate PR in the weblinks project.
I did not do menu items in this patch as it looks we may have to check for other aspects in the code.