Feeling Lucky
?
[#32649] - Update default.php
- Closed
- 12 Mar 2021
- Medium
- Build: staging
- # 32649
- Diff
- saumyasarkar11:patch-4
User tests: Successful: Unsuccessful:
Pull Request for Issue # .
Summary of Changes
Check for the emptiness of $urls on lines 98 and 132 has been removed since the variable $urls can never be empty.
Testing Instructions
Create an article and check whether it is properly displayed or not with its url.
Actual result BEFORE applying this Pull Request
Expected result AFTER applying this Pull Request
Documentation Changes Required
None
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Front End com_content |
saumyasarkar11
- comment
- 12 Mar 2021
This change is not correct.
$urlscan be empty. You didn't tested using the URL fields on an article. In the screenshot you see the fields Link A, Link B and Link C, that are the URL fields:
Got it. Thanks for the feedback. Will look into it.
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-03-12 08:54:18 |
| Closed_By | ⇒ | saumyasarkar11 | |
| Labels |
Added:
?
|
||
This change is not correct.
$urlscan be empty. You didn't tested using the URL fields on an article. In the screenshot you see the fields Link A, Link B and Link C, that are the URL fields: