Skip to content

Adding default accesslevel to menu item#11638

Merged
wilsonge merged 1 commit intojoomla:stagingfrom
Bakual:FixDefaultMenuAccesslevel
Aug 19, 2016
Merged

Adding default accesslevel to menu item#11638
wilsonge merged 1 commit intojoomla:stagingfrom
Bakual:FixDefaultMenuAccesslevel

Conversation

@Bakual
Copy link
Copy Markdown
Contributor

@Bakual Bakual commented Aug 17, 2016

Pull Request for Issue #11317 .

Summary of Changes

Removing useless default value for the accesslevel in the form xml and adding default value from global config to new menu items.

Testing Instructions

  • Change the default accesslevel to something else than "Published" (eg to "Registered") in the global configuration.
  • Create a new menu item and check that the accesslevel is what you set in the global configuration.

Documentation Changes Required

Nope.

@infograf768
Copy link
Copy Markdown
Member

I have tested this item ✅ successfully on db648fe


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

1 similar comment
@jeckodevelopment
Copy link
Copy Markdown
Member

I have tested this item ✅ successfully on db648fe


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

@jeckodevelopment
Copy link
Copy Markdown
Member

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Aug 17, 2016
@Bakual Bakual added this to the Joomla 3.6.3 milestone Aug 17, 2016
@wilsonge wilsonge merged commit e6911f8 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
@Bakual Bakual deleted the FixDefaultMenuAccesslevel branch August 19, 2016 19:13
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