Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change "protect" state of Plugin editors-xtd "Button - Article" #5297

Closed
wants to merge 2 commits into from
Closed

Change "protect" state of Plugin editors-xtd "Button - Article" #5297

wants to merge 2 commits into from

Conversation

pe7er
Copy link
Contributor

@pe7er pe7er commented Dec 3, 2014

with this update for existing Joomla installations.

See issue: http://issues.joomla.org/tracker/joomla-cms/4875

@Bakual
Copy link
Contributor

Bakual commented Dec 3, 2014

Can you do the query based on the name (and maybe also type, element and folder) instead of by id? Depending on enviroment and how they updated the site the ID may actually be different. Using the name should however work in all cases.

Update query for MySQL, based on the name, type, element and folder (instead of by id).
@pe7er
Copy link
Contributor Author

pe7er commented Dec 3, 2014

Done for MySQL db1e217

@Hackwar
Copy link
Member

Hackwar commented Dec 4, 2014

why are there 2 PRs for this that are identical as far as I can see? I mean #5298

@brianteeman
Copy link
Contributor

Look again. They are for different db
On 4 Dec 2014 19:56, "Hannes Papenberg" notifications@github.com wrote:

why are there 2 PRs for this that are identical as far as I can see? I
mean #5298 #5298


Reply to this email directly or view it on GitHub
#5297 (comment).

@pe7er
Copy link
Contributor Author

pe7er commented Dec 4, 2014

Yeah, sorry for the confusion @Hackwar.
I should have added the database type in the subject...

@waader
Copy link
Contributor

waader commented Dec 5, 2014

@test works.

I tested this one as I have described in #5298.

@anibalsanchez
Copy link
Contributor

@test OK, all buttons can be disabled


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

@zero-24
Copy link
Contributor

zero-24 commented Jan 15, 2015

RTC Thanks!


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

@brianteeman brianteeman added the RTC This Pull Request is Ready To Commit label Jan 16, 2015
@roland-d roland-d added this to the Joomla! 3.4.0 milestone Jan 17, 2015
@roland-d roland-d closed this in a15b28e Jan 17, 2015
@zero-24 zero-24 removed the RTC This Pull Request is Ready To Commit label Oct 14, 2015
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.

None yet

9 participants