Skip to content

Solve PHP Notice when saving as copy a category#11664

Merged
wilsonge merged 1 commit intojoomla:stagingfrom
infograf768:notice_assoc_cat
Aug 19, 2016
Merged

Solve PHP Notice when saving as copy a category#11664
wilsonge merged 1 commit intojoomla:stagingfrom
infograf768:notice_assoc_cat

Conversation

@infograf768
Copy link
Copy Markdown
Member

@infograf768 infograf768 commented Aug 19, 2016

Patch similar to #11647 for category

Summary of Changes

There is a PHP Notice (check php error logs) when saving as copy a category.

Testing Instructions

  1. In multilingual joomla install with associations open any category and save as copy. Now check your php log and you will have a php notice there PHP Notice: Undefined index: associations in ROOT/administrator/components/com_categories/models/category.php on line 575
  2. Apply patch
  3. Repeat step 1. No PHP Notice

Other components use the default in JModelAdmin.

@andrepereiradasilva @jreys @jeckodevelopment @alikon

@jreys
Copy link
Copy Markdown
Contributor

jreys commented Aug 19, 2016

I have tested this item ✅ successfully on

No warning :)


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

@brianteeman
Copy link
Copy Markdown
Contributor

I have tested this item ✅ successfully on bbd7945


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

@brianteeman
Copy link
Copy Markdown
Contributor

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Aug 19, 2016
@brianteeman brianteeman added this to the Joomla 3.6.3 milestone Aug 19, 2016
@wilsonge wilsonge merged commit c1a1772 into joomla:staging Aug 19, 2016
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Aug 19, 2016
@infograf768 infograf768 deleted the notice_assoc_cat branch August 19, 2016 14:30
ggppdk pushed a commit to ggppdk/joomla-cms that referenced this pull request Aug 19, 2016
roland-d pushed a commit to roland-d/joomla-cms that referenced this pull request Sep 11, 2016
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.

5 participants