[4.0] Error when changing status of tagged content items #30466
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
I have tested this item ✅ successfully on 07f56cc This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/30466. |
1 similar comment
|
I have tested this item ✅ successfully on 07f56cc This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/30466. |
|
RTC This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/30466. |
|
Thanks! |
drmenzelit
pushed a commit
to drmenzelit/joomla-cms
that referenced
this pull request
Aug 31, 2020
* Correct column name * Fix changing UCM state
infograf768
added a commit
that referenced
this pull request
Sep 4, 2020
* First step to make newsflash horizontal
* [4.0] Fix Alert double error display in Installation
* seperating multiple messages with border top
* Increase minimum length
* Fix meter displaying complete message prematurely
* Add strenthmeter attribute to installation
* Change meter values to display green when password requirements are met
* [4.0] {Cassiopea] Implementing Password length/meter
* [4.0] Fix Fieldset description display (#30435)
* [4.0] Url Language Code is not a prefix (#30440)
* [4.0] URL LANGUAGE CODE is not a prefix
* modified desc as suggested
* [4.0] Implementing display of fieldset descriptions for fieldset
children
* Taking off description
* scss cs
* use @Quy suggestion
* file cs
* [4.0] Media field fix (#30455)
* fix
* typo, (well nothing new)
* Allow media fields in subforms
- Cleanup Media field
- Expose initilazation method of Bootstrpa Modlas to a Joomla.Bootstrap object
- Initilise Modlas on the lifecycle of the Custom Element
* Paleolithic js mode only in the legacy…
* CS
* Proper check
* Pffff
* Update joomla-field-media.w-c.es6.js
* [4.0] Convert icon used for jgrid defaults to fas fa- (#30464)
* make icon-circle use fa-circle
* allows for including icon- in call
* move fa-fw to last class to fit convention (#30469)
* [4.0] Long labels wrapping (#30474)
When we have fields displayed in columns with the label displayed above the input there is a width limit of 240px on the label.
This results in long labels wrapping when it is not needed as the column is wider than 240px.
In english we dont see this is many places but the easiest is in the __configure edit screen_ fields
This simple PR changes the label width to 100% when the fields are displayed in columns like this.
As this is an scss change you will need to `node build.js --compile-css` in order to test
### Before
### After
* Fix mod_articles_latest (#30459)
* [4.0] Add a parameter for "back-to-top" button in Cassiopeia (#30441)
* [4.0] Add a parameter for back-to-top button in Cassiopeia
Added a parameter to show / hide the back-to-top button in Cassiopeia.
Currently the footer appears when a module has been published on position "footer". And the back-to-top only appears if the footer is visible. Since the footer is part of the grid, it is not easy to move the back-to-top outside the grid.
Now it is possible to decide to have a back-to-top or not. If yes, the footer will be rendered showing the button, independently if there is a module in the position footer.
It is also possible to show only modules (switch back-to-top off) or modules and back-to-top (switch on).
* Added smooth scrolling
Add smooth scrolling taking in account the preferences of the user (reduced motion).
Improve language file.
* Correct height and position of back-to-top button
* Correct code style
* Correct code style 2
* Change smooth scrolling behavior to take in account reduced motion preferences
* Language file corrected, language string changed to match others
* Update language/en-GB/tpl_cassiopeia.ini
Link instead button
Co-authored-by: Brian Teeman <[email protected]>
* Update templates/cassiopeia/scss/blocks/_global.scss
Comment for smooth scroll
Co-authored-by: Brian Teeman <[email protected]>
Co-authored-by: Brian Teeman <[email protected]>
* fix improper family name. (#30481)
* [4.0] update helptoc (#30490)
* [4.0] update helptoc
Removes the string for the expired cache page that no longer exists and removes it from the index displayed on the left hand side in the help page
* api
* [4.0] Add new permissions-policy to the HTTPHeaders Plugin (#30491)
* add permissions-policy
* add Permissions-Policy to the dropdown
* alpha order
* [4.0] Error when changing status of tagged content items (#30466)
* Correct column name
* Fix changing UCM state
* [4.0] br tag (#30503)
* [4.0] br tag
In Joomla 4 we use `<br>` not `<br />`
* bad grep
* [4.0] Remove old search component (#30506)
* [4.0] Remove old search component
The regular search component is not in J4
This pr removes reference to it in the help system
`administrator/index.php?option=com_admin&view=help`
* revert
* [4.0] Help Dashboard (#30508)
* [4.0] Help Dashboard
This PR makes multiple changes to the help dashboard. In order of importance they are.
- Created a new section "Start Here"
- Moved "Joomla Help" to the "Start Here" section to give it greater priority as it really is the first place you should look for help
- Removed target=_blank from "Joomla Help"
- Moved "Documentation Wiki" from "Resources" to the top of "Additional Help"
* deduplicate
* compass icon
* Moved css for mod_articles_news to media folder
User webAsset Manager to load specific css for module
* Move web asset call to horizontal.php
Co-authored-by: Jean-Marie Simonet <[email protected]>
Co-authored-by: Quy <[email protected]>
Co-authored-by: dGrammatiko <[email protected]>
Co-authored-by: Bear <[email protected]>
Co-authored-by: Brian Teeman <[email protected]>
Co-authored-by: Christiane Maier-Stadtherr <[email protected]>
Co-authored-by: Tobias Zulauf <[email protected]>
Co-authored-by: SharkyKZ <[email protected]>
sakiss
pushed a commit
to sakiss/joomla-cms
that referenced
this pull request
Oct 16, 2020
* Correct column name * Fix changing UCM state
sakiss
pushed a commit
to sakiss/joomla-cms
that referenced
this pull request
Oct 16, 2020
* First step to make newsflash horizontal
* [4.0] Fix Alert double error display in Installation
* seperating multiple messages with border top
* Increase minimum length
* Fix meter displaying complete message prematurely
* Add strenthmeter attribute to installation
* Change meter values to display green when password requirements are met
* [4.0] {Cassiopea] Implementing Password length/meter
* [4.0] Fix Fieldset description display (joomla#30435)
* [4.0] Url Language Code is not a prefix (joomla#30440)
* [4.0] URL LANGUAGE CODE is not a prefix
* modified desc as suggested
* [4.0] Implementing display of fieldset descriptions for fieldset
children
* Taking off description
* scss cs
* use @Quy suggestion
* file cs
* [4.0] Media field fix (joomla#30455)
* fix
* typo, (well nothing new)
* Allow media fields in subforms
- Cleanup Media field
- Expose initilazation method of Bootstrpa Modlas to a Joomla.Bootstrap object
- Initilise Modlas on the lifecycle of the Custom Element
* Paleolithic js mode only in the legacy…
* CS
* Proper check
* Pffff
* Update joomla-field-media.w-c.es6.js
* [4.0] Convert icon used for jgrid defaults to fas fa- (joomla#30464)
* make icon-circle use fa-circle
* allows for including icon- in call
* move fa-fw to last class to fit convention (joomla#30469)
* [4.0] Long labels wrapping (joomla#30474)
When we have fields displayed in columns with the label displayed above the input there is a width limit of 240px on the label.
This results in long labels wrapping when it is not needed as the column is wider than 240px.
In english we dont see this is many places but the easiest is in the __configure edit screen_ fields
This simple PR changes the label width to 100% when the fields are displayed in columns like this.
As this is an scss change you will need to `node build.js --compile-css` in order to test
### Before
### After
* Fix mod_articles_latest (joomla#30459)
* [4.0] Add a parameter for "back-to-top" button in Cassiopeia (joomla#30441)
* [4.0] Add a parameter for back-to-top button in Cassiopeia
Added a parameter to show / hide the back-to-top button in Cassiopeia.
Currently the footer appears when a module has been published on position "footer". And the back-to-top only appears if the footer is visible. Since the footer is part of the grid, it is not easy to move the back-to-top outside the grid.
Now it is possible to decide to have a back-to-top or not. If yes, the footer will be rendered showing the button, independently if there is a module in the position footer.
It is also possible to show only modules (switch back-to-top off) or modules and back-to-top (switch on).
* Added smooth scrolling
Add smooth scrolling taking in account the preferences of the user (reduced motion).
Improve language file.
* Correct height and position of back-to-top button
* Correct code style
* Correct code style 2
* Change smooth scrolling behavior to take in account reduced motion preferences
* Language file corrected, language string changed to match others
* Update language/en-GB/tpl_cassiopeia.ini
Link instead button
Co-authored-by: Brian Teeman <[email protected]>
* Update templates/cassiopeia/scss/blocks/_global.scss
Comment for smooth scroll
Co-authored-by: Brian Teeman <[email protected]>
Co-authored-by: Brian Teeman <[email protected]>
* fix improper family name. (joomla#30481)
* [4.0] update helptoc (joomla#30490)
* [4.0] update helptoc
Removes the string for the expired cache page that no longer exists and removes it from the index displayed on the left hand side in the help page
* api
* [4.0] Add new permissions-policy to the HTTPHeaders Plugin (joomla#30491)
* add permissions-policy
* add Permissions-Policy to the dropdown
* alpha order
* [4.0] Error when changing status of tagged content items (joomla#30466)
* Correct column name
* Fix changing UCM state
* [4.0] br tag (joomla#30503)
* [4.0] br tag
In Joomla 4 we use `<br>` not `<br />`
* bad grep
* [4.0] Remove old search component (joomla#30506)
* [4.0] Remove old search component
The regular search component is not in J4
This pr removes reference to it in the help system
`administrator/index.php?option=com_admin&view=help`
* revert
* [4.0] Help Dashboard (joomla#30508)
* [4.0] Help Dashboard
This PR makes multiple changes to the help dashboard. In order of importance they are.
- Created a new section "Start Here"
- Moved "Joomla Help" to the "Start Here" section to give it greater priority as it really is the first place you should look for help
- Removed target=_blank from "Joomla Help"
- Moved "Documentation Wiki" from "Resources" to the top of "Additional Help"
* deduplicate
* compass icon
* Moved css for mod_articles_news to media folder
User webAsset Manager to load specific css for module
* Move web asset call to horizontal.php
Co-authored-by: Jean-Marie Simonet <[email protected]>
Co-authored-by: Quy <[email protected]>
Co-authored-by: dGrammatiko <[email protected]>
Co-authored-by: Bear <[email protected]>
Co-authored-by: Brian Teeman <[email protected]>
Co-authored-by: Christiane Maier-Stadtherr <[email protected]>
Co-authored-by: Tobias Zulauf <[email protected]>
Co-authored-by: SharkyKZ <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #30465.
Summary of Changes
Corrects column name and removes misplaced context check.
Testing Instructions
Create an article.
Add some tags.
Go to articles list in backend.
Click on an icon to publish/unpublish the article.
Actual result BEFORE applying this Pull Request
Expected result AFTER applying this Pull Request
No errors.
Documentation Changes Required
No.