-
-
Notifications
You must be signed in to change notification settings - Fork 2
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
[pull] 4.0-dev from joomla:4.0-dev #128
Conversation
…when both belong to same vendor. (#34954) * [4.0] (#34910) Updating or uninstalling one library no longer removes another when both belong to same vendor. ### Summary of Changes Check if there are any other children before removing vendor folder. ### Testing instructions As described in #34910 * Added blank line for PHP code style error. * Removed the empty arrays for the last 2 parameters of Folder::folders() so the defaults are used As suggestion made in pull comment. * Removed unnecessary optional parameters from Folder::folders()
* [4.0] remove file check * cleanup debug
When navigating with the keyboard a focus outline is applied to the tabs However as can be seen in this enlargement the outline is cut on the right. This PR adjusts the z-index to prevent that
* Remove wrong information from code comment * More precise comment about discovered extensions
.map((directory) => this._normalizeItem(directory)); | ||
const files = data.filter((item) => (item.type === 'file')) | ||
// eslint-disable-next-line no-underscore-dangle | ||
.map((file) => this._normalizeItem(file)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected parentheses around single function argument having a body with no curly braces arrow-parens
const directories = data.filter((item) => (item.type === 'dir')) | ||
// eslint-disable-next-line no-underscore-dangle | ||
.map((directory) => this._normalizeItem(directory)); | ||
const files = data.filter((item) => (item.type === 'file')) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected parentheses around single function argument having a body with no curly braces arrow-parens
_normalizeArray(data) { | ||
const directories = data.filter((item) => (item.type === 'dir')) | ||
// eslint-disable-next-line no-underscore-dangle | ||
.map((directory) => this._normalizeItem(directory)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected parentheses around single function argument having a body with no curly braces arrow-parens
*/ | ||
// eslint-disable-next-line no-underscore-dangle | ||
_normalizeArray(data) { | ||
const directories = data.filter((item) => (item.type === 'dir')) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected parentheses around single function argument having a body with no curly braces arrow-parens
*/ | ||
fire(event, data = null) { | ||
if (this.events[event]) { | ||
this.events[event].forEach((fn) => fn(data)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected parentheses around single function argument having a body with no curly braces arrow-parens
Co-authored-by: Thomas Hunziker <[email protected]>
* Allow the translation bot to recompile the htmls * Update cleanup-media.es6.js * Update cleanup-media.es6.js * Update build/build-modules-js/init/cleanup-media.es6.js Co-authored-by: Brian Teeman <[email protected]> * Update cleanup-media.es6.js Co-authored-by: Brian Teeman <[email protected]>
* [4.0] One more * Update cleanup-media.es6.js
…5510) * Fix pre-update check wrong update required information * Fix wrong code comment * Simplify code * Revert unrelated change for collection URLs * Optimize code * Use array_values for the intersect * PHPCS
return val; | ||
}); | ||
}, | ||
install: (Vue) => Vue.mixin({ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected parentheses around single function argument having a body with no curly braces arrow-parens
|
||
const Translate = { | ||
// Translate from Joomla text | ||
translate: (key) => Joomla.Text._(key, key), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected parentheses around single function argument having a body with no curly braces arrow-parens
*/ | ||
export const toggleBrowserItemSelect = (context, payload) => { | ||
const item = payload; | ||
const isSelected = context.state.selectedItems.some((selected) => selected.path === item.path); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected parentheses around single function argument having a body with no curly braces arrow-parens
* @returns {Array|files|{}|FileList|*} | ||
*/ | ||
export const getSelectedDirectoryFiles = (state) => state.files | ||
.filter((file) => (file.directory === state.selectedDirectory)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected parentheses around single function argument having a body with no curly braces arrow-parens
* | ||
* @returns {Array|files|{}|FileList|*} | ||
*/ | ||
export const getSelectedDirectoryFiles = (state) => state.files |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected parentheses around single function argument having a body with no curly braces arrow-parens
*/ | ||
[types.UNSELECT_BROWSER_ITEM]: (state, payload) => { | ||
const item = payload; | ||
state.selectedItems.splice(state.selectedItems.findIndex( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected newline after '(' function-paren-newline
if (item.type === 'dir') { | ||
state.directories.splice(state.directories.findIndex( | ||
(directory) => directory.path === item.path, | ||
), 1); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected newline before ')' function-paren-newline
// Delete dir | ||
if (item.type === 'dir') { | ||
state.directories.splice(state.directories.findIndex( | ||
(directory) => directory.path === item.path, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected parentheses around single function argument having a body with no curly braces arrow-parens
|
||
// Delete dir | ||
if (item.type === 'dir') { | ||
state.directories.splice(state.directories.findIndex( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected newline after '(' function-paren-newline
if (item.type === 'file') { | ||
state.files.splice(state.files.findIndex( | ||
(file) => file.path === item.path, | ||
), 1); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected newline before ')' function-paren-newline
// Delete file | ||
if (item.type === 'file') { | ||
state.files.splice(state.files.findIndex( | ||
(file) => file.path === item.path, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected parentheses around single function argument having a body with no curly braces arrow-parens
|
||
// Delete file | ||
if (item.type === 'file') { | ||
state.files.splice(state.files.findIndex( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected newline after '(' function-paren-newline
const index = state.files.findIndex((file) => (file.path === oldPath)); | ||
state.files.splice(index, 1, item); | ||
} else { | ||
const index = state.directories.findIndex((directory) => (directory.path === oldPath)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected parentheses around single function argument having a body with no curly braces arrow-parens
const { item } = payload; | ||
const { oldPath } = payload; | ||
if (item.type === 'file') { | ||
const index = state.files.findIndex((file) => (file.path === oldPath)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected parentheses around single function argument having a body with no curly braces arrow-parens
// Update the relation to the parent directory | ||
state.directories.splice(parentDirectoryIndex, | ||
1, | ||
{ ...parentDirectory, directories: [...parentDirectory.directories, directory.path] }); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Expected a newline before ')' function-paren-newline
*/ | ||
[types.UPLOAD_SUCCESS]: (state, payload) => { | ||
const file = payload; | ||
const isNew = (!state.files.some((existing) => (existing.path === file.path))); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected parentheses around single function argument having a body with no curly braces arrow-parens
state.directories | ||
.splice(parentDirectoryIndex, | ||
1, | ||
{ ...parentDirectory, files: [...parentDirectory.files, file.path] }); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Expected a newline before ')' function-paren-newline
// Update the relation to the parent directory | ||
if (parentDirectoryIndex !== -1) { | ||
state.directories | ||
.splice(parentDirectoryIndex, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Expected a newline after '(' function-paren-newline
const parentDirectory = state.directories | ||
.find((directory) => (directory.path === file.directory)); | ||
const parentDirectoryIndex = state.directories.indexOf(parentDirectory); | ||
let index = state.files.findIndex((existing) => (existing.path === file.path)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected parentheses around single function argument having a body with no curly braces arrow-parens
// eslint-disable-next-line no-shadow | ||
function addFile(state, file) { | ||
const parentDirectory = state.directories | ||
.find((directory) => (directory.path === file.directory)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected parentheses around single function argument having a body with no curly braces arrow-parens
state.directories | ||
.splice(parentDirectoryIndex, | ||
1, | ||
{ ...parentDirectory, directories: [...parentDirectory.directories, directory.path] }); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Expected a newline before ')' function-paren-newline
// Update the relation to the parent directory | ||
if (parentDirectoryIndex !== -1) { | ||
state.directories | ||
.splice(parentDirectoryIndex, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Expected a newline after '(' function-paren-newline
const parentDirectory = state.directories | ||
.find((existing) => (existing.path === directory.directory)); | ||
const parentDirectoryIndex = state.directories.indexOf(parentDirectory); | ||
let index = state.directories.findIndex((existing) => (existing.path === directory.path)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected parentheses around single function argument having a body with no curly braces arrow-parens
// eslint-disable-next-line no-shadow | ||
function addDirectory(state, directory) { | ||
const parentDirectory = state.directories | ||
.find((existing) => (existing.path === directory.directory)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected parentheses around single function argument having a body with no curly braces arrow-parens
// Add the sub directories and files | ||
directory.directories = state.directories | ||
.filter((existing) => existing.directory === directory.path) | ||
.map((existing) => existing.path); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected parentheses around single function argument having a body with no curly braces arrow-parens
|
||
// Add the sub directories and files | ||
directory.directories = state.directories | ||
.filter((existing) => existing.directory === directory.path) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected parentheses around single function argument having a body with no curly braces arrow-parens
* @param path | ||
*/ | ||
function createDirectoryStructureFromPath(path) { | ||
const exists = state.directories.some((existing) => (existing.path === path)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected parentheses around single function argument having a body with no curly braces arrow-parens
displayName: disk.displayName, | ||
drives: getDrives(disk.adapterNames, disk.name), | ||
})); | ||
const defaultDisk = loadedDisks.find((disk) => disk.drives.length > 0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected parentheses around single function argument having a body with no curly braces arrow-parens
}; | ||
|
||
// Load disks from options | ||
const loadedDisks = options.providers.map((disk) => ({ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected parentheses around single function argument having a body with no curly braces arrow-parens
*/ | ||
const getDrives = (adapterNames, provider) => { | ||
const drives = []; | ||
adapterNames.map((name) => drives.push({ root: `${provider}-${name}:/`, displayName: name })); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unexpected parentheses around single function argument having a body with no curly braces arrow-parens
This updates the skipto js to the newly released 4.1.2 ### Version 4.1.2 Added aria-busy="true" attribute to menu element when SkipTo is initialized and being updated with new menu items to support validators looking for required menu items for the menu role. Added the optional aria-controls attribute to button element to reference the id of the menu element as defined in the W3C ARIA Authoring practices for menu button pattern. ### Version 4.1.1 Removed aria-describedby from button, since screen readers read the accesskey information.
If you do not have the track session metadata option enabled in global configuration then the admin module for Logged Users displays an appropriate message. This PR simply styles the module output correctly to be consistent
* [J4] Adds support for custom class on subforms via xml * [J4] Adds support for custom class on subforms via xml
See Commits and Changes for more details.
Created by pull[bot]
Can you help keep this open source service alive? 💖 Please sponsor : )