? Pending

User tests: Successful: Unsuccessful:

avatar saumyasarkar11
saumyasarkar11
19 Mar 2021

Pull Request for Issue #32725.

Summary of Changes

  1. && $this->item->pagination removed from https://github.com/joomla/joomla-cms/blob/4.0.0-beta7/components/com_content/tmpl/article/default.php#L43.

  2. Comment and brackets around OR removed from https://github.com/joomla/joomla-cms/blob/4.0.0-beta7/components/com_content/tmpl/article/default.php#L49-L51.

Documentation Changes Required

None.

avatar saumyasarkar11 saumyasarkar11 - open - 19 Mar 2021
avatar saumyasarkar11 saumyasarkar11 - change - 19 Mar 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 19 Mar 2021
Category Front End com_content
avatar saumyasarkar11 saumyasarkar11 - change - 19 Mar 2021
The description was changed
avatar saumyasarkar11 saumyasarkar11 - edited - 19 Mar 2021
avatar saumyasarkar11 saumyasarkar11 - change - 19 Mar 2021
The description was changed
avatar saumyasarkar11 saumyasarkar11 - edited - 19 Mar 2021
avatar joomdonation
joomdonation - comment - 19 Mar 2021

Your PR contains more changes than needed. You should make PR base on the code of latest 4.0-dev branch:

  • You reverted the changes in this merged PR #32713
  • As stated in the issue, the comment should be there, it should not be removed
avatar saumyasarkar11 saumyasarkar11 - change - 19 Mar 2021
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2021-03-19 10:43:47
Closed_By saumyasarkar11
Labels Added: ?
avatar saumyasarkar11 saumyasarkar11 - close - 19 Mar 2021
avatar ReLater
ReLater - comment - 19 Mar 2021

As stated in the issue, the comment should be there, it should not be removed

That's wrong. In the issue.

I think it's there from an older version, can be removed

avatar saumyasarkar11 saumyasarkar11 - change - 19 Mar 2021
Status Closed New
Closed_Date 2021-03-19 10:43:47
Closed_By saumyasarkar11
avatar saumyasarkar11 saumyasarkar11 - change - 19 Mar 2021
Status New Pending
avatar saumyasarkar11 saumyasarkar11 - reopen - 19 Mar 2021
avatar saumyasarkar11
saumyasarkar11 - comment - 19 Mar 2021

@ReLater please look through this pull request.

avatar ReLater ReLater - test_item - 19 Mar 2021 - Tested successfully
avatar ReLater
ReLater - comment - 19 Mar 2021

I have tested this item successfully on 7c10476


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/32745.

avatar ReLater ReLater - test_item - 19 Mar 2021 - Tested unsuccessfully
avatar ReLater
ReLater - comment - 19 Mar 2021

I have tested this item ? unsuccessfully on 7c10476


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/32745.

avatar saumyasarkar11
saumyasarkar11 - comment - 19 Mar 2021

I have tested this item unsuccessfully on 7c10476

This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/32745.

Can you just elaborate on this.....Would be helpful

avatar ReLater
ReLater - comment - 19 Mar 2021

Sorry! Confusion on my side.
That one is correct but closed. #32746

This one here (32745) is wrong because you included changes that are already merged with this pr: #32713

avatar saumyasarkar11
saumyasarkar11 - comment - 19 Mar 2021

Sorry! Confusion on my side.
That one is correct but closed. #32746

This one here (32745) is wrong because you included changes that are already merged with this pr: #32713

So i'm opening that one.

avatar saumyasarkar11 saumyasarkar11 - change - 19 Mar 2021
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2021-03-19 11:30:51
Closed_By saumyasarkar11
avatar saumyasarkar11 saumyasarkar11 - close - 19 Mar 2021

Add a Comment

Login with GitHub to post a comment