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

[pull] 4.0-dev from joomla:4.0-dev #128

Closed
wants to merge 4,859 commits into from
Closed

Conversation

pull[bot]
Copy link

@pull pull bot commented Aug 9, 2021

See Commits and Changes for more details.


Created by pull[bot]

Can you help keep this open source service alive? 💖 Please sponsor : )

BrainforgeUK and others added 21 commits August 2, 2021 10:35
…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
@pull pull bot added the ⤵️ pull label Aug 9, 2021
.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));
Copy link

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'))
Copy link

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));
Copy link

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'))
Copy link

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));
Copy link

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

@pull pull bot added the merge-conflict Resolve conflicts manually label Aug 9, 2021
ReLater and others added 9 commits September 9, 2021 18:13
Co-authored-by: Thomas Hunziker <werbemails@bakual.ch>
* 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 <brian@teeman.net>

* Update cleanup-media.es6.js

Co-authored-by: Brian Teeman <brian@teeman.net>
* [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({
Copy link

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),
Copy link

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);
Copy link

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));
Copy link

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
Copy link

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(
Copy link

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);
Copy link

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,
Copy link

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(
Copy link

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);
Copy link

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,
Copy link

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(
Copy link

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));
Copy link

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));
Copy link

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] });
Copy link

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)));
Copy link

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] });
Copy link

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,
Copy link

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));
Copy link

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));
Copy link

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] });
Copy link

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,
Copy link

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));
Copy link

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));
Copy link

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);
Copy link

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)
Copy link

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));
Copy link

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
Copy link

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) => ({
Copy link

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 }));
Copy link

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

dgrammatiko and others added 16 commits September 12, 2021 10:05
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.
Fixes bug #35454 (Debug plugin: Debug bar doesn't display logs and deprecated tabs by default, while setting's default is ON)
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⤵️ pull merge-conflict Resolve conflicts manually
Projects
None yet
Development

Successfully merging this pull request may close these issues.