Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Testing setting wp version #3584

Draft
wants to merge 85 commits into
base: master
Choose a base branch
from
Draft

Testing setting wp version #3584

wants to merge 85 commits into from

Conversation

Camwyn
Copy link
Member

@Camwyn Camwyn commented Feb 21, 2025

No description provided.

sdokus and others added 30 commits February 10, 2025 16:06
…age-template-override-path-correction

[ET-2296] Correct template override path for My Tickets page
sdokus and others added 27 commits February 18, 2025 15:21
…-booking-fees-to-emails

[ET-2297] Add Booking Fees to Confirmation Emails
…m:the-events-calendar/event-tickets into fix/ET-2309-prevent-fatal-attenddee-model
* Modify pup config and GitHub zip workflow.
* Organize composer.json and add code to prevent composer from timing out when running pup build commands.
* Add distfiles
* update-browserslist (bonus)
…vent-tickets into fix/ET-2309-prevent-fatal-attenddee-model
…m:the-events-calendar/event-tickets into fix/ET-2309-prevent-fatal-attenddee-model
…-fatal-attenddee-model

Prevent fatal error for Attendee Models with Invalid Orders
coupled with removing the update --force in the yml files.
@Camwyn Camwyn added the hold Status: on hold–do not proceed with other status items. label Feb 21, 2025
@Camwyn Camwyn self-assigned this Feb 21, 2025
date_i18n( tribe_get_date_format( true ), strtotime( esc_attr( $order['purchase_time'] ) ) )
esc_attr( $purchaser_name ),
'<a href="mailto:' . esc_url( $purchaser_email ) . '">' . esc_html( $purchaser_email ) . '</a>',
$purchase_time ? date_i18n( tribe_get_date_format( true ), strtotime( esc_attr( $purchase_time ) ) ) : __( 'Unknown Time (invalid order)', 'event-tickets' )
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[phpcs] reported by reviewdog 🐶
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not '$purchase_time'

date_i18n( tribe_get_date_format( true ), strtotime( esc_attr( $order['purchase_time'] ) ) )
esc_attr( $purchaser_name ),
'<a href="mailto:' . esc_url( $purchaser_email ) . '">' . esc_html( $purchaser_email ) . '</a>',
$purchase_time ? date_i18n( tribe_get_date_format( true ), strtotime( esc_attr( $purchase_time ) ) ) : __( 'Unknown Time (invalid order)', 'event-tickets' )
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[phpcs] reported by reviewdog 🐶
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found 'date_i18n'.

date_i18n( tribe_get_date_format( true ), strtotime( esc_attr( $order['purchase_time'] ) ) )
esc_attr( $purchaser_name ),
'<a href="mailto:' . esc_url( $purchaser_email ) . '">' . esc_html( $purchaser_email ) . '</a>',
$purchase_time ? date_i18n( tribe_get_date_format( true ), strtotime( esc_attr( $purchase_time ) ) ) : __( 'Unknown Time (invalid order)', 'event-tickets' )
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[phpcs] reported by reviewdog 🐶
StellarWP.XSS.EscapeOutput.OutputNotEscaped
Expected next thing to be an escaping function (see Codex for 'Data Validation'), not ')'

date_i18n( tribe_get_date_format( true ), strtotime( esc_attr( $order['purchase_time'] ) ) )
esc_attr( $purchaser_name ),
'<a href="mailto:' . esc_url( $purchaser_email ) . '">' . esc_html( $purchaser_email ) . '</a>',
$purchase_time ? date_i18n( tribe_get_date_format( true ), strtotime( esc_attr( $purchase_time ) ) ) : __( 'Unknown Time (invalid order)', 'event-tickets' )
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[phpcs] reported by reviewdog 🐶
WordPress.Security.EscapeOutput.OutputNotEscaped
All output should be run through an escaping function (see the Security sections in the WordPress Developer Handbooks), found '__'.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hold Status: on hold–do not proceed with other status items.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants