[#17175] - Prepare also the intro text in the article details view
- Fixed in Code Base
- 20 Aug 2017
- Medium
- Build: staging
- # 17175
- Diff
- Digital-Peak:cf/process-intro-text
- Success hound No violations found. Woof! Details
- Success continuous-integration/drone/pr the build was successful Details
- Success continuous-integration/travis-ci/pr The Travis CI build passed Details
- Success continuous-integration/appveyor/pr AppVeyor build succeeded Details
- Pending JTracker/HumanTestResults Human Test Results: 1 Successful 0 Failed. Details
User tests: Successful: Unsuccessful:
Pull Request for Issue #15751.
Summary of Changes
Triggers the prepare event also on the intro text in the article details view. So are plugins triggered on the intro text as well.
Testing Instructions
- Insert Field in Article
- Article Options - Set Linked Titles to Yes, Show Unauthorised Links to Yes, Show Intro Text to Hide,
- Set Article Access to Registered
- Create any custom field with access right to public and insert in the article before read more (intro text)
- Create category blog menu item
- Check front end as non logged in user.
Expected result
Clicking on article title should show inserted field value.
Actual result
Clicking on article title show field name as (field --}
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Front End com_content |
| Title |
|
||||||
| Title |
|
||||||
@esskay303 please test this PR.
Hello...Thanks for your response and extremely sorry for not responding in time. I have coped the site to another server. I am sorry but would like to know what is the best way to test it. Is it okay if I copy components/com_content/views/article/view.html.php this file to the test server and check the behaviour? Sorry once again for the basics but new here...
Follow the instructions here
https://docs.joomla.org/Component_Patchtester_for_Testers
or here
https://brian.teeman.net/joomla/a-dummies-guide-to-joomla-bug-testing
Hi Brian...Thanks for your guidance. It seems that the issue is fixed with this patch. However, I will be able to do in depth testing only by tomorrow and see whether everything is working as expected or not.
Meanwhile, what should be done for the applied patch? Should I keep it as it is i.e. applied or I need to revert it?
I can confirm that this has fixed my issue. Will this be a part of the next release?
@esskay303 can you please submit the successfully Test at https://issues.joomla.org/tracker/joomla-cms/17175?
Now this PR has 2 succefully Tests and maintainers decide if it goes in next Release (Hint: Feature Freeze was a few Days ago so normally it can't come with next Release).
I have tested this item
Tested successfully and working...
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/17175.
| Status | Pending | ⇒ | Ready to Commit |
RTC after two successful tests.
mbabker
- | Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-08-20 15:38:43 |
| Closed_By | ⇒ | mbabker | |
| Labels |
Added:
?
|
||
I have tested this item✅ successfully on 1493c9e
Hint: In Menu > Options
Show Unauthorised Linksset toUse Article Settingsthen Article is displayed > @laoneo can you please update Test Instructions?This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/17175.