Skip to content

Commit

Permalink
Move deprecation warnings to debug
Browse files Browse the repository at this point in the history
Signed-off-by: Simon L <szaimen@e.mail.de>
  • Loading branch information
szaimen committed Jun 24, 2023
1 parent 4f66692 commit b10e7e7
Show file tree
Hide file tree
Showing 11 changed files with 17 additions and 17 deletions.
6 changes: 3 additions & 3 deletions apps/files/js/filelist.js
Original file line number Diff line number Diff line change
Expand Up @@ -666,7 +666,7 @@
* @param {string} [tabId] optional tab id to select
*/
showDetailsView: function(fileName, tabId) {
console.warn('showDetailsView is deprecated! Use OCA.Files.Sidebar.activeTab. It will be removed in nextcloud 20.');
OC.debug && console.warn('showDetailsView is deprecated! Use OCA.Files.Sidebar.activeTab. It will be removed in nextcloud 20.');
this._updateDetailsView(fileName);
if (tabId) {
OCA.Files.Sidebar.setActiveTab(tabId);
Expand Down Expand Up @@ -3879,7 +3879,7 @@
* Register a tab view to be added to all views
*/
registerTabView: function(tabView) {
console.warn('registerTabView is deprecated! It will be removed in nextcloud 20.');
OC.debug && console.warn('registerTabView is deprecated! It will be removed in nextcloud 20.');
const enabled = tabView.canDisplay || undefined
if (tabView.id) {
OCA.Files.Sidebar.registerTab(new OCA.Files.Sidebar.Tab({
Expand All @@ -3905,7 +3905,7 @@
* Register a detail view to be added to all views
*/
registerDetailView: function(detailView) {
console.warn('registerDetailView is deprecated! It will be removed in nextcloud 20.');
OC.debug && console.warn('registerDetailView is deprecated! It will be removed in nextcloud 20.');
if (detailView.el) {
OCA.Files.Sidebar.registerSecondaryView(detailView)
}
Expand Down
4 changes: 2 additions & 2 deletions apps/files/js/files.js
Original file line number Diff line number Diff line change
Expand Up @@ -281,7 +281,7 @@
* @deprecated used OCA.Files.FileList.generatePreviewUrl instead
*/
generatePreviewUrl: function(urlSpec) {
console.warn('DEPRECATED: please use generatePreviewUrl() from an OCA.Files.FileList instance');
OC.debug && console.warn('DEPRECATED: please use generatePreviewUrl() from an OCA.Files.FileList instance');
return OCA.Files.App.fileList.generatePreviewUrl(urlSpec);
},

Expand All @@ -290,7 +290,7 @@
* @deprecated used OCA.Files.FileList.lazyLoadPreview instead
*/
lazyLoadPreview : function(path, mime, ready, width, height, etag) {
console.warn('DEPRECATED: please use lazyLoadPreview() from an OCA.Files.FileList instance');
OC.debug && console.warn('DEPRECATED: please use lazyLoadPreview() from an OCA.Files.FileList instance');
return FileList.lazyLoadPreview({
path: path,
mime: mime,
Expand Down
2 changes: 1 addition & 1 deletion apps/files_sharing/src/services/ExternalLinkActions.js
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@ export default class ExternalLinkActions {
* @return {boolean}
*/
registerAction(action) {
console.warn('OCA.Sharing.ExternalLinkActions is deprecated, use OCA.Sharing.ExternalShareAction instead')
OC.debug && console.warn('OCA.Sharing.ExternalLinkActions is deprecated, use OCA.Sharing.ExternalShareAction instead')

if (typeof action === 'object' && action.icon && action.name && action.url) {
this._state.actions.push(action)
Expand Down
2 changes: 1 addition & 1 deletion core/src/OC/appsettings.js
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ import { generateFilePath } from "@nextcloud/router"
* @deprecated 17.0.0 this method is unused and will be removed with Nextcloud 18
*/
export const appSettings = args => {
console.warn('OC.appSettings is deprecated and will be removed with Nextcloud 18')
OC.debug && console.warn('OC.appSettings is deprecated and will be removed with Nextcloud 18')

if (typeof args === 'undefined' || typeof args.appid === 'undefined') {
throw {
Expand Down
2 changes: 1 addition & 1 deletion core/src/OC/capabilities.js
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,6 @@ import { getCapabilities as realGetCapabilities } from '@nextcloud/capabilities'
* @since 14.0.0
*/
export const getCapabilities = () => {
console.warn('OC.getCapabilities is deprecated and will be removed in Nextcloud 21. See @nextcloud/capabilities')
OC.debug && console.warn('OC.getCapabilities is deprecated and will be removed in Nextcloud 21. See @nextcloud/capabilities')
return realGetCapabilities()
}
4 changes: 2 additions & 2 deletions core/src/OC/legacy-loader.js
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ const loadedStyles = []
* @deprecated 16.0.0 Use OCP.Loader.loadScript
*/
export const addScript = (app, script, ready) => {
console.warn('OC.addScript is deprecated, use OCP.Loader.loadScript instead')
OC.debug && console.warn('OC.addScript is deprecated, use OCP.Loader.loadScript instead')

let deferred
const path = generateFilePath(app, 'js', script + '.js')
Expand All @@ -64,7 +64,7 @@ export const addScript = (app, script, ready) => {
* @deprecated 16.0.0 Use OCP.Loader.loadStylesheet
*/
export const addStyle = (app, style) => {
console.warn('OC.addStyle is deprecated, use OCP.Loader.loadStylesheet instead')
OC.debug && console.warn('OC.addStyle is deprecated, use OCP.Loader.loadStylesheet instead')

const path = generateFilePath(app, 'css', style + '.css')
if (loadedStyles.indexOf(path) === -1) {
Expand Down
4 changes: 2 additions & 2 deletions core/src/OC/util.js
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,7 @@ export default {
*/
formatDate(timestamp, format) {
if (window.TESTING === undefined) {
console.warn('OC.Util.formatDate is deprecated and will be removed in Nextcloud 21. See @nextcloud/moment')
OC.debug && console.warn('OC.Util.formatDate is deprecated and will be removed in Nextcloud 21. See @nextcloud/moment')
}
format = format || 'LLL'
return moment(timestamp).format(format)
Expand All @@ -136,7 +136,7 @@ export default {
*/
relativeModifiedDate(timestamp) {
if (window.TESTING === undefined) {
console.warn('OC.Util.relativeModifiedDate is deprecated and will be removed in Nextcloud 21. See @nextcloud/moment')
OC.debug && console.warn('OC.Util.relativeModifiedDate is deprecated and will be removed in Nextcloud 21. See @nextcloud/moment')
}
const diff = moment().diff(moment(timestamp))
if (diff >= 0 && diff < 45000) {
Expand Down
2 changes: 1 addition & 1 deletion core/src/OCA/search.js
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ export default class Search {
* @deprecated 20.0.0, will be removed in Nextcloud 22
*/
constructor() {
console.warn('OCA.Search is deprecated. Please use the unified search API instead')
OC.debug && console.warn('OCA.Search is deprecated. Please use the unified search API instead')
}

}
4 changes: 2 additions & 2 deletions core/src/Polyfill/tooltip.js
Original file line number Diff line number Diff line change
Expand Up @@ -31,13 +31,13 @@ $.prototype.tooltip = (function(tooltip) {
} catch (ex) {
if (ex instanceof TypeError && config === 'destroy') {
if (window.TESTING === undefined) {
console.error('Deprecated call $.tooltip(\'destroy\') has been deprecated and should be removed')
OC.debug && console.warn('Deprecated call $.tooltip(\'destroy\') has been deprecated and should be removed')
}
return tooltip.call(this, 'dispose')
}
if (ex instanceof TypeError && config === 'fixTitle') {
if (window.TESTING === undefined) {
console.error('Deprecated call $.tooltip(\'fixTitle\') has been deprecated and should be removed')
OC.debug && console.warn('Deprecated call $.tooltip(\'fixTitle\') has been deprecated and should be removed')
}
return tooltip.call(this, '_fixTitle')
}
Expand Down
2 changes: 1 addition & 1 deletion core/src/globals.js
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ import { getToken as getRequestToken } from './OC/requesttoken.js'

const warnIfNotTesting = function() {
if (window.TESTING === undefined) {
console.warn.apply(console, arguments)
OC.debug && console.warn.apply(console, arguments)
}
}

Expand Down
2 changes: 1 addition & 1 deletion core/src/jquery/placeholder.js
Original file line number Diff line number Diff line change
Expand Up @@ -140,7 +140,7 @@ const toRgb = (s) => {
}

String.prototype.toRgb = function() {
console.warn('String.prototype.toRgb is deprecated! It will be removed in Nextcloud 22.')
OC.debug && console.warn('String.prototype.toRgb is deprecated! It will be removed in Nextcloud 22.')

return toRgb(this)
}
Expand Down

0 comments on commit b10e7e7

Please sign in to comment.