Merge branch 'main' into feature/73-som-component

This commit is contained in:
Justin Foell
2021-01-29 12:16:56 -06:00
5 changed files with 39 additions and 34 deletions
+12 -1
View File
@@ -109,11 +109,22 @@ class WPStrava_Activity {
* @since 2.3.2
*/
public function get_activity_link( $activity_id, $text, $title = '' ) {
if ( WPStrava::get_instance()->settings->no_link ) {
if ( $this->is_rest_request() || WPStrava::get_instance()->settings->no_link ) {
return $text;
}
$url = esc_url( self::ACTIVITIES_URL . $activity_id );
$title_attr = $title ? " title='" . esc_attr( $title ) . "'" : '';
return "<a href='{$url}'{$title_attr}>{$text}</a>";
}
/**
* Check if rest request to skip link rendering in block editor.
*
* @return boolean
* @author Justin Foell <justin@foell.org>
* @since 2.5.1
*/
private function is_rest_request() {
return defined( 'REST_REQUEST' ) && REST_REQUEST;
}
}
+12 -14
View File
@@ -98,20 +98,11 @@ class WPStrava_ActivityRenderer {
<div class="activity-details-table-units">' . $strava_som->get_pace_label() . '</div>
</td>';
$max_speed = '<td data-label="' . __( 'Max Speed', 'wp-strava' ) . '">
<div>' . $strava_som->pace( $activity_details->max_speed ) . '</div>
<div class="activity-details-table-info">' . $strava_som->pace( $activity_details->max_speed ) . '</div>
<div class="activity-details-table-units">' . $strava_som->get_pace_label() . '</div>
</td>';
break;
case WPStrava_ActivityType::TYPE_GROUP_SPEED:
$avg_speed = '<td data-label="' . __( 'Average Speed', 'wp-strava' ) . '">
<div class="activity-details-table-info">' . $strava_som->speed( $activity_details->average_speed ) . '</div>
<div class="activity-details-table-units">' . $strava_som->get_speed_label() . '</div>
</td>';
$max_speed = '<td data-label="' . __( 'Max Speed', 'wp-strava' ) . '">
<div class="activity-details-table-info">' . $strava_som->speed( $activity_details->max_speed ) . '</div>
<div class="activity-details-table-units">' . $strava_som->get_speed_label() . '</div>
</td>';
break;
case WPStrava_ActivityType::TYPE_GROUP_PACE:
case WPStrava_ActivityType::TYPE_GROUP_WATER:
$avg_speed = '<td data-label="' . __( 'Average Speed', 'wp-strava' ) . '">
<div class="activity-details-table-info">' . $strava_som->swimpace( $activity_details->average_speed ) . '</div>
<div class="activity-details-table-units">' . $strava_som->get_swimpace_label() . '</div>
@@ -121,9 +112,16 @@ class WPStrava_ActivityRenderer {
<div class="activity-details-table-units">' . $strava_som->get_swimpace_label() . '</div>
</td>';
break;
case WPStrava_ActivityType::TYPE_GROUP_SPEED:
default:
$avg_title = '';
$max_title = '';
$avg_speed = '<td data-label="' . __( 'Average Speed', 'wp-strava' ) . '">
<div class="activity-details-table-info">' . $strava_som->speed( $activity_details->average_speed ) . '</div>
<div class="activity-details-table-units">' . $strava_som->get_speed_label() . '</div>
</td>';
$max_speed = '<td data-label="' . __( 'Max Speed', 'wp-strava' ) . '">
<div class="activity-details-table-info">' . $strava_som->speed( $activity_details->max_speed ) . '</div>
<div class="activity-details-table-units">' . $strava_som->get_speed_label() . '</div>
</td>';
break;
}
+6 -14
View File
@@ -1,6 +1,6 @@
<?php
/**
* ActivityType [activitytype].
* ActivityType [activitytype].
* @package WPStrava
*/
@@ -45,22 +45,18 @@ class WPStrava_ActivityType {
const TYPE_WORKOUT = 'Workout';
const TYPE_YOGA = 'Yoga';
const TYPE_DEFAULT = self::TYPE_RIDE;
private static $water_types = array( self::TYPE_SWIM );
private static $pace_types = array( self::TYPE_CANOEING, self::TYPE_HIKE, self::TYPE_RUN, self::TYPE_SNOWSHOE, self::TYPE_VIRTUALRUN, self::TYPE_WALK );
private static $speed_types = array( self::TYPE_ALPINESKI, self::TYPE_BACKCOUNTRYSKI, self::TYPE_EBIKERIDE, self::TYPE_ELLIPTICAL, self::TYPE_HANDCYCLE, self::TYPE_ICESKATE, self::TYPE_INLINESKATE, self::TYPE_KAYAKING, self::TYPE_KITESURF, self::TYPE_NORDICSKI, self::TYPE_RIDE, self::TYPE_ROCKCLIMBING, self::TYPE_ROLLERSKI, self::TYPE_ROWING, self::TYPE_SNOWBOARD, self::TYPE_STAIRSTEPPER, self::TYPE_STANDUPPADDLING, self::TYPE_SURFING, self::TYPE_VIRTUALRIDE, self::TYPE_WHEELCHAIR, self::TYPE_WINDSURF );
private static $water_types = array( self::TYPE_SWIM );
const TYPE_GROUP_WATER = 'water';
const TYPE_GROUP_PACE = 'pace';
const TYPE_GROUP_WATER = 'water';
const TYPE_GROUP_SPEED = 'speed';
const TYPE_GROUP_OTHER = 'other';
/**
* Get the type of activity.
* Get the type of activity - defaults to 'speed'.
*
* @param string $type Type provided by Strava.
* @return string Type group (water/pace/speed/other).
* @return string Type group (pace/water/speed).
* @author Sebastian Erb <mail@sebastianerb.com>
* @since 1.7.0
*/
@@ -70,15 +66,11 @@ class WPStrava_ActivityType {
return self::TYPE_GROUP_PACE;
}
if ( in_array( $type, self::$speed_types, true ) ) {
return self::TYPE_GROUP_SPEED;
}
if ( in_array( $type, self::$water_types, true ) ) {
return self::TYPE_GROUP_WATER;
}
return self::TYPE_GROUP_OTHER;
return self::TYPE_GROUP_SPEED;
}
}