Fix drupal coding standard for php function strpos.

From strpos to Unicode::strpos Using Drupal\Component\Utility\Unicode.

CommentFileSizeAuthor
#7 interdiff_5-7.txt4.62 KBkkalashnikov
#7 3196480-7.patch8.9 KBkkalashnikov
#5 3196480-5.patch3.81 KBshefali99
#2 3196480-2.patch3.73 KBdeepakkumar14

Issue fork xmlsitemap-3196480

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

deepakkumar14 created an issue. See original summary.

deepakkumar14’s picture

deepakkumar14 CreditAttribution: deepakkumar14 as a volunteer and at gai Technologies Pvt Ltd for gai Technologies Pvt Ltd commented
Issue summary: View changes
Status: Active » Needs review
FileSize
3196480-2.patch3.73 KB

Added patch for this.

samir_shukla made their first commit to this issue’s fork.

Status: Needs review » Needs work

The last submitted patch, 2: 3196480-2.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

shefali99’s picture

shefali99 CreditAttribution: shefali99 at OpenSense Labs for DrupalFit commented
Status: Needs work » Needs review
FileSize
3196480-5.patch3.81 KB

#2 patch failed for D10, resolved this issue . Need review

Status: Needs review » Needs work

The last submitted patch, 5: 3196480-5.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

kkalashnikov’s picture

kkalashnikov CreditAttribution: kkalashnikov as a volunteer and at Material for Drupal India Association commented
FileSize
3196480-7.patch8.9 KB
interdiff_5-7.txt4.62 KB

chaitanyadessai made their first commit to this issue’s fork.

chaitanyadessai opened merge request !42