ReplaceText should allow empty replacement strings on the command line
Closed, ResolvedPublicFeature
Actions

Assigned To

Authored By

	Buovjaga
	Oct 12 2021, 5:34 PM

Project Tags

Referenced Files

None

Subscribers

• Aklapper

Buovjaga

Description

If you leave the replacement string empty with command line use, you just get the message "You have to specify replacement text". Would be nice to have parity with the web UI.

Details

Related Changes in Gerrit:

	Subject	Author	Repo	Branch	Lines +/-
	replaceAll.php: Allow empty values as target	Fomafix	mediawiki/extensions/ReplaceText	master	+2 -2

Customize query in gerrit

Event Timeline

Buovjaga created this task.Oct 12 2021, 5:34 PM

Restricted Application added a subscriber: • Aklapper. · View Herald TranscriptOct 12 2021, 5:34 PM

@Buovjaga: Thanks for reporting this. For future reference, please use the feature request form (linked from the top of the task creation page) to create feature requests. Thanks.

Fomafix claimed this task.Sep 8 2022, 12:15 PM

Change 830851 had a related patch set uploaded (by Fomafix; author: Fomafix):

[mediawiki/extensions/ReplaceText@master] replaceAll.php: Allow empty values as target

https://gerrit.wikimedia.org/r/830851

gerritbot added a project: Patch-For-Review.Sep 8 2022, 12:15 PM

Change 830851 merged by jenkins-bot:

[mediawiki/extensions/ReplaceText@master] replaceAll.php: Allow empty values as target

https://gerrit.wikimedia.org/r/830851

Maintenance_bot removed a project: Patch-For-Review.Sep 8 2022, 1:30 PM

Buovjaga closed this task as Resolved.Sep 9 2022, 3:40 PM

ReplaceText should allow empty replacement strings on the command lineClosed, ResolvedPublicFeatureActions

Description

Details

Event Timeline

ReplaceText should allow empty replacement strings on the command line
Closed, ResolvedPublicFeature
Actions