[#9474] - Add Microdata to popular articles - hard coded values
- Fixed in Code Base
- 25 Mar 2016
- Medium
- Build: staging
- # 9474
- Diff
- photodude:patch-3
- Success continuous-integration/travis-ci/pr The Travis CI build passed Details
- Success JTracker/HumanTestResults Human Test Results: 2 Successful 0 Failed. Details
User tests: Successful: Unsuccessful:
Pull Request to Add Microdata to popular articles.
Summary of Changes
Adds hard coded Microdata to popular articles following the format established in the latest articles module
Testing Instructions
- Add patch https://docs.joomla.org/Testing_Joomla!_patches
- Add a popular articles module to a view
- Check that microdata values are applied using a microdata validation tool such as https://developers.google.com/structured-data/testing-tool/
Additional Comments
With the JMicrodata PR's #8934 and #8933 in review, this PR is proposed to add the hard coded Microdata values so that schema.org markup can be added in the short term.
The does not replace #8934, nor is it an alternative. This PR is just a solution to put in place the missing Microdata values while the change to use the JMicrodata library is in Review.
Votes
| Status | New | ⇒ | Pending |
| Labels |
Added:
?
|
||
| Category | ⇒ | Modules |
I have tested this item 
unsuccessfully on 0e90632
breaks links as noted above
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9474.
I have tested this item unsuccessfully on 0e90632
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9474.
This PR has received new commits.
CC: @brianteeman, @franz-wohlkoenig
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9474.
I have tested this item 
successfully on ca0b09f
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9474.
I have tested this item successfully on localhost
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9474.
I have tested this item successfully on ca0b09f
The patch works fine!
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9474.
@brianteeman Can this be set to RTC now?
wilsonge
- | Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-03-25 15:11:58 |
| Closed_By | ⇒ | wilsonge |
Titles not linked
Without Patch you see linked Titles in Module "Articles - Most Read":
With Patch unlinked Titles:
Testing Tools Failure
Without Patch you see no Module Failure (Failures only in "Article")
With Patch there are Failures: