[#19950] - Fix for duplicate url check bug introduced by #19734 and support utf8…
- Fixed in Code Base
- 25 Mar 2018
- Medium
- Build: staging
- # 19950
- Diff
- tonypartridge:staging-redirects-manual-utf8
- Pending continuous-integration/drone/pr this build is pending Details
User tests: Successful: Unsuccessful:
Fix for duplicate url check bug introduced by #19734 and support utf8 on old_urls.
Couldn't find a solution to handle this within mysql. So a simple foreach handles it perfectly fine.
@infograf768 can you please test?
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration com_redirect |
I have tested this item
Great!
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/19950.
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/19950.
| Labels |
Added:
?
|
||
Done
On 20 Mar 2018, 22:46 +0000, Quy notifications@github.com, wrote:
@Quy commented on this pull request.
In administrator/components/com_redirect/tables/link.php:
$xid = (int) $db->loadResult();if ($xid && $xid != (int) $this->id)
foreach ($urls AS $url)Change AS to as
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.
| Status | Pending | ⇒ | Ready to Commit |
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/19950.
mbabker
- | Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2018-03-25 14:59:36 |
| Closed_By | ⇒ | mbabker | |
| Labels |
Added:
?
|
||
I have tested this item✅ successfully on dc2021a
Great!
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/19950.