added $deleteUnusedRoutes parameter to Route::refreshRoutes()

Author: johny011409

controllers/PermissionController.php

@@ -117,12 +117,13 @@ public function actionSetChildRoutes($id)
 	 * Add new routes and remove unused (for example if module or controller was deleted)
 	 *
 	 * @param string $id
+	 * @param null   $deleteUnused
 	 *
 	 * @return \yii\web\Response
 	 */
-	public function actionRefreshRoutes($id)
+	public function actionRefreshRoutes($id, $deleteUnused = null)
 	{
-		Route::refreshRoutes();
+		Route::refreshRoutes($deleteUnused !== null);
 
 		return $this->redirect(['view', 'id'=>$id]);
 	}

models/rbacDB/Route.php

@@ -105,29 +105,42 @@ public static function isSubRoute($route, $candidate)
 	}
 
 	/**
-	 * Refresh list of all routes from controllers, modules, etc
+	 * Refresh list of all routes from controllers, modules, etc.
+	 *
+	 * If $deleteUnusedRoutes is true, than all routes that are not longer exists in this application
+	 * (for example if you delete some controller or module) will be deleted.
+	 *
+	 * $deleteUnusedRoutes = false is recommended for application with "advanced" structure, because frontend
+	 * and backend have they own set of routes.
+	 *
+	 * @param bool $deleteUnusedRoutes
 	 */
-	public static function refreshRoutes()
+	public static function refreshRoutes($deleteUnusedRoutes = true)
 	{
 		$allRoutes = AuthHelper::getRoutes();
 
 		$currentRoutes = ArrayHelper::map(Route::find()->asArray()->all(), 'name', 'name');
 
 		$toAdd = array_diff(array_keys($allRoutes), array_keys($currentRoutes));
-		$toRemove = array_diff(array_keys($currentRoutes), array_keys($allRoutes));
-
 
 		foreach ($toAdd as $addItem)
 		{
 			Route::create($addItem);
 		}
 
-		if ( $toRemove )
+		$toRemove = false;
+		if ( $deleteUnusedRoutes )
 		{
-			Route::deleteAll(['in', 'name', $toRemove]);
+			$toRemove = array_diff(array_keys($currentRoutes), array_keys($allRoutes));
+
+			if ( $toRemove )
+			{
+				Route::deleteAll(['in', 'name', $toRemove]);
+			}
 		}
 
-		if ( $toAdd OR $toRemove )
+
+		if ( $toAdd || $toRemove )
 		{
 			Yii::$app->cache->delete('__commonRoutes');
 		}

views/permission/view.php

@@ -83,15 +83,26 @@
 				<strong>
 					<span class="glyphicon glyphicon-th"></span> Routes
 
+					<?= Html::a(
+						'Refresh routes (and delete unused)',
+						['refresh-routes', 'id'=>$item->name, 'deleteUnused'=>1],
+						[
+							'class' => 'btn btn-default btn-sm pull-right',
+							'style'=>'margin-top:-5px; text-transform:none;',
+							'data-confirm'=>UserManagementModule::t('back', 'Routes that are not exists in this application will be deleted. Do not recommended for application with "advanced" structure, because frontend and backend have they own set of routes.'),
+						]
+					) ?>
+
 					<?= Html::a(
 						'Refresh routes',
 						['refresh-routes', 'id'=>$item->name],
 						[
 							'class' => 'btn btn-default btn-sm pull-right',
-							'style'=>'margin-top:-5px',
+							'style'=>'margin-top:-5px; text-transform:none;',
 						]
 					) ?>
 
+
 				</strong>
 			</div>