Remove unused getRingIndex from Doughnut (#8521)

chartjs · Feb 24, 2021 · 12e9020 · 12e9020
1 parent 8796a1b
commit 12e9020
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 14 deletions.
diff --git a/docs/docs/getting-started/v3-migration.md b/docs/docs/getting-started/v3-migration.md
@@ -303,6 +303,7 @@ The following properties and methods were removed:
 * `DatasetController.addElementAndReset`
 * `DatasetController.createMetaData`
 * `DatasetController.createMetaDataset`
+* `DoughnutController.getRingIndex`
 
 #### Removed from Elements
 

diff --git a/src/controllers/controller.doughnut.js b/src/controllers/controller.doughnut.js
@@ -60,19 +60,6 @@ export default class DoughnutController extends DatasetController {
     }
   }
 
-  // Get index of the dataset in relation to the visible datasets. This allows determining the inner and outer radius correctly
-  getRingIndex(datasetIndex) {
-    let ringIndex = 0;
-
-    for (let j = 0; j < datasetIndex; ++j) {
-      if (this.chart.isDatasetVisible(j)) {
-        ++ringIndex;
-      }
-    }
-
-    return ringIndex;
-  }
-
   /**
 	 * @private
 	 */

diff --git a/types/index.esm.d.ts b/types/index.esm.d.ts
@@ -314,7 +314,6 @@ export interface DoughnutController extends DatasetController {
 	readonly offsetX: number;
 	readonly offsetY: number;
 
-	getRingIndex(datasetIndex: number): number;
 	calculateTotal(): number;
 	calculateCircumference(value: number): number;
 }