Merge pull request #56089 from nextcloud/chore/remove-oc-set-get

chore: remove `OC.set` and `OC.get`
This commit is contained in:
Ferdinand Thiessen 2025-10-30 23:13:06 +01:00 committed by GitHub
commit f4e8cd13a4
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
6 changed files with 6 additions and 61 deletions

View file

@ -1,46 +0,0 @@
/**
* SPDX-FileCopyrightText: 2019 Nextcloud GmbH and Nextcloud contributors
* SPDX-License-Identifier: AGPL-3.0-or-later
*/
/**
*
* @param context
*/
export function get(context) {
return (name) => {
const namespaces = name.split('.')
const tail = namespaces.pop()
for (let i = 0; i < namespaces.length; i++) {
context = context[namespaces[i]]
if (!context) {
return false
}
}
return context[tail]
}
}
/**
* Set a variable by name
*
* @param {string} context context
* @return {Function} setter
* @deprecated 19.0.0 use https://lodash.com/docs#set
*/
export function set(context) {
return (name, value) => {
const namespaces = name.split('.')
const tail = namespaces.pop()
for (let i = 0; i < namespaces.length; i++) {
if (!context[namespaces[i]]) {
context[namespaces[i]] = {}
}
context = context[namespaces[i]]
}
context[tail] = value
return value
}
}

View file

@ -49,7 +49,6 @@ import { currentUser, getCurrentUser } from './currentuser.js'
import { debug } from './debug.js'
import Dialogs from './dialogs.js'
import EventSource from './eventsource.js'
import { get, set } from './get_set.js'
import {
getHost,
getHostName,
@ -234,14 +233,6 @@ export default {
* @deprecated 19.0.0 use `generateUrl` from https://www.npmjs.com/package/@nextcloud/router
*/
generateUrl,
/**
* @deprecated 19.0.0 use https://lodash.com/docs#get
*/
get: get(window),
/**
* @deprecated 19.0.0 use https://lodash.com/docs#set
*/
set: set(window),
/**
* @deprecated 19.0.0 use `getRootUrl` from https://www.npmjs.com/package/@nextcloud/router
*/

4
dist/core-login.js vendored

File diff suppressed because one or more lines are too long

File diff suppressed because one or more lines are too long

4
dist/core-main.js vendored

File diff suppressed because one or more lines are too long

File diff suppressed because one or more lines are too long