Patch to clean up theme template files to use consistent PHP formatting.

CommentFileSizeAuthor
#3 calendar-theme_template_cleanup-2581039-3.patch30.38 KBron_s
#2 calendar-theme_template_cleanup-2581039-2.patch26.63 KBron_s
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ron_s created an issue. See original summary.

ron_s’s picture

ron_s CreditAttribution: ron_s commented
FileSize
calendar-theme_template_cleanup-2581039-2.patch26.63 KB

See attached.

ron_s’s picture

ron_s CreditAttribution: ron_s commented
FileSize
calendar-theme_template_cleanup-2581039-3.patch30.38 KB

An updated version with some additional cleanups.

joelpittet’s picture

joelpittet
Primary language English
Location Vancouver
CreditAttribution: joelpittet commented
Status: Needs review » Reviewed & tested by the community

Thanks ron, I scanned the patch and these are all quite straight forward and inline with the coding standards.

Neslee Canil Pinto’s picture

Neslee Canil Pinto
he/him
Primary language English
CreditAttribution: Neslee Canil Pinto at Specbee commented
Status: Reviewed & tested by the community » Closed (outdated)
joelpittet’s picture

joelpittet
Primary language English
Location Vancouver
CreditAttribution: joelpittet at The University of British Columbia commented
Status: Closed (outdated) » Fixed

A bit of a reroll but I committed the code thanks @ron_s

Status: Fixed » Closed (fixed)

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

ron_s’s picture

ron_s CreditAttribution: ron_s commented

Looks like a bunch of these were missed in the reroll, but I'm not going to write another patch for it.