Skip to content

Commit

Permalink
Fix PHPStan issues.
Browse files Browse the repository at this point in the history
  • Loading branch information
remcotolsma committed May 7, 2024
1 parent 766eb58 commit 1e8e640
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 4 deletions.
4 changes: 2 additions & 2 deletions src/Extension.php
Original file line number Diff line number Diff line change
Expand Up @@ -1065,7 +1065,7 @@ private function maybe_update_pronamic_subscription() {
return;
}

$subscription = \get_pronamic_subscription( $subscription_id );
$subscription = \get_pronamic_subscription( (int) $subscription_id );

if ( null === $subscription ) {
return;
Expand All @@ -1079,7 +1079,7 @@ private function maybe_update_pronamic_subscription() {

$source_id = $subscription->get_source_id();

$rcp_membership = \rcp_get_membership( $source_id );
$rcp_membership = \rcp_get_membership( (int) $source_id );

if ( false === $rcp_membership ) {
return;
Expand Down
10 changes: 8 additions & 2 deletions src/SubscriptionUpdater.php
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ public function update_pronamic_subscription() {
$rcp_membership = $this->rcp_membership;
$pronamic_subscription = $this->pronamic_subscription;

$rcp_membership_level = \rcp_get_membership_level( $rcp_membership->get_object_id() );
$rcp_membership_level = \rcp_get_membership_level( (int) $rcp_membership->get_object_id() );

if ( false === $rcp_membership_level ) {
throw new \Exception( 'Cannot find the Restrict Content membership level for the Restrict Content membership to be updated.' );
Expand Down Expand Up @@ -84,10 +84,16 @@ public function update_pronamic_subscription() {
);

$initial_phase->set_total_periods( 1 );

$initial_phase_end_date = $initial_phase->get_end_date();

if ( null === $initial_phase_end_date ) {
throw new \Exception( 'The initial subscription phase has no end date, this should not happen.' );
}

$regular_phase = new SubscriptionPhase(
$pronamic_subscription,
$initial_phase->get_end_date(),
$initial_phase_end_date,
new SubscriptionInterval( 'P' . $rcp_membership_level->get_duration() . LengthUnit::to_core( $rcp_membership_level->get_duration_unit() ) ),
new Money( $rcp_membership->get_recurring_amount(), $rcp_membership->get_currency() )
);
Expand Down

0 comments on commit 1e8e640

Please sign in to comment.