Problem/Motivation

Since #2554889: Remove SafeMarkup::set() from the codebase is committed, we should remove the method.

Proposed resolution

Remove\Replace the calls to the method.

Remaining tasks

User interface changes

Data model changes

CommentFileSizeAuthor
#2 remove_set-2563007-2.patch1.61 KBgiancarlosotelo

Comments

giancarlosotelo created an issue. See original summary.

giancarlosotelo’s picture

giancarlosotelo commented
Status: Active » Needs review
StatusFileSize
new remove_set-2563007-2.patch1.61 KB

Tested locally and works well.

LKS90’s picture

LKS90 commented
Status: Needs review » Reviewed & tested by the community

Looks good to go, tested it and looked for more occurrences (there were none).

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 2: remove_set-2563007-2.patch, failed testing.

berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Needs work » Fixed

Looks good, thanks.

Status: Fixed » Closed (fixed)

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

Status: Closed (fixed) » Needs work

The last submitted patch, 2: remove_set-2563007-2.patch, failed testing.

giancarlosotelo’s picture

giancarlosotelo commented
Status: Needs work » Closed (fixed)