Quantcast
Channel: Issues for Drupal core
Viewing all articles
Browse latest Browse all 297377

StatisticsViewsResult should only deal in integer values

$
0
0

Problem/Motivation

This is a follow-up to #2500521: Upgrade path for Statistics 7.x and 6.x where we identified that StatisticsViewsResult was (sometimes) returning strings when it should only ever return integers.

Proposed resolution

Cast the inputs to integer in the constructor.

Remaining tasks

Write patch.
Review patch.
etc.
Removes @todo from Drupal\Tests\statistics\Kernel\Migrate\d6\MigrateNodeCounterTest and Drupal\Tests\statistics\Kernel\Migrate\d7\MigrateNodeCounterTest (assuming #2500521: Upgrade path for Statistics 7.x and 6.x lands first).

User interface changes

API changes

Data model changes


Viewing all articles
Browse latest Browse all 297377

Trending Articles



<script src="https://jsc.adskeeper.com/r/s/rssing.com.1596347.js" async> </script>