Fix error in most popular posts diagram showing multiple pages with the same title (#42)

Author: patrickrobrecht

CHANGELOG.md

@@ -7,6 +7,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## Version 2.x
 
+### Version 2.6.2
+* Bugfix: Error in most popular posts diagram showing multiple pages with the same title
+
 ### Version 2.6.1
 * Update tooltip library (switch to more recent fork)
 * Fix issues on WordPress multisite installations (set capabilities and links on the content page)

views/content.php

@@ -78,7 +78,7 @@ class="<?php eefstatify_echo_tab_class( $selected_post_type === $post_type ); ?>
 	</form>
 	<?php if ( count( $views_per_post ) === 0 ) { ?>
 	<p><?php esc_html_e( 'No data available.', 'extended-evaluation-for-statify' ); ?></p>
-<?php } else { ?>
+	<?php } else { ?>
 	<section>
 		<?php
 		$legend = [];
@@ -98,8 +98,8 @@ class="<?php eefstatify_echo_tab_class( $selected_post_type === $post_type ); ?>
 				'#chart-popular-content',
 				[
 					<?php
-					foreach ( $views_per_post_for_diagram as $post ) {
-						echo "['" . esc_js( eefstatify_get_post_title_from_url( $post['url'] ) ) . "'," . esc_js( $post['count'] ) . '],';
+					foreach ( $views_per_post_for_diagram as $index => $post ) {
+						echo "['" . esc_js( $index ) . "'," . esc_js( $post['count'] ) . '],';
 					}
 					?>
 				]