Skip to content

[4.0] Remove show_associations field of association is not enabled#23501

Merged
wilsonge merged 7 commits intojoomla:4.0-devfrom
joomdonation:patch-1
Jan 14, 2019
Merged

[4.0] Remove show_associations field of association is not enabled#23501
wilsonge merged 7 commits intojoomla:4.0-devfrom
joomdonation:patch-1

Conversation

@joomdonation
Copy link
Contributor

Pull Request for Issue # .

Summary of Changes

This simple PR removes the field show_associations field of association is not enabled. It causes fatal error while saving article when I tested PR #12414 and @Hackwar had to add a check (which I think not necessary) to prevent the error.

Plus, it also removes some unused variables/lines of code which can be seen easily if you use a good IDE like phpstorm.

Testing Instructions

Apply patch, try to save article and confirm nothing broken. Maybe code review should be enough.

@infograf768
Copy link
Member

I have tested this item ✅ successfully on 802acf8

ok on review.


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

@astridx
Copy link
Contributor

astridx commented Jan 12, 2019

I applied this patch and I could create and save an article.

I see no change in the options tab of com_content.
The field show_associations is shown when the Plugin System - Language Filter is active and it is not shown when the Plugin System - Language Filter is not active - with and without this patch.

Perhaps we can delete the line

in this PR, too.

@astridx
Copy link
Contributor

astridx commented Jan 12, 2019

I have tested this item ✅ successfully on 88d5f95


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

@wilsonge
Copy link
Contributor

@zero-24 RIPS please

@zero-24
Copy link
Contributor

zero-24 commented Jan 14, 2019

@zero-24 RIPS please

RIPS seams to be fine?

@wilsonge wilsonge merged commit 635473b into joomla:4.0-dev Jan 14, 2019
@wilsonge
Copy link
Contributor

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants