Updated: Comment #0

Problem/Motivation

Noted in #1952394-62: Add configuration translation user interface module in core

+++ b/core/modules/config_translation/config_translation.moduleundefined
@@ -0,0 +1,382 @@
+ */
+function config_translation_help($path, $arg) {

@catch

This will conflict with the $arg patch if it gets in. Since the second argument isn't used, it could be left off - then if that patch does go in this won't break and vice versa.

arg issue: #788900: Deprecate and remove usages of arg()

Proposed resolution

remove arg

Remaining tasks

User interface changes

No.

API changes

No.

CommentFileSizeAuthor
#1 config_translation_remove_arg.2035449-1.patch443 bytesLinL
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

YesCT’s picture

Issue summary: View changes

added right arg issue, catch pinged to me

LinL’s picture

Status: Active » Needs review
FileSize
443 bytes
vijaycs85’s picture

Status: Needs review » Fixed

Committed e363245 and pushed to 8.x-1.x. Thanks!

Automatically closed -- issue fixed for 2 weeks with no activity.

Anonymous’s picture

Issue summary: View changes

updating in other location.