Merge pull request #3134 from joaorjose/dev

Typecast model Id so that urls don't break when using MongoDB object Id'...

Merge pull request #3134 from joaorjose/dev
f210ec82 · Carsten Brandt · a42b80ec · 831be938 · f210ec82
Commit f210ec82 authored Apr 17, 2014 by Carsten Brandt
Hide whitespace changes
Inline Side-by-side

Showing with 10 additions and 2 deletions

Generator.php extensions/gii/generators/crud/Generator.php +10 -2

No files found.
--- a/extensions/gii/generators/crud/Generator.php
+++ b/extensions/gii/generators/crud/Generator.php
@@ -433,11 +433,19 @@ class Generator extends \yii\gii\Generator
        $class = $this->modelClass;
        $pks = $class::primaryKey();
        if (count($pks) === 1) {
-            return "'id' => \$model->{$pks[0]}";
+            if (is_subclass_of($class, 'yii\mongodb\ActiveRecord')) {
+                return "'id' => (string)\$model->{$pks[0]}";
+            } else {
+                return "'id' => \$model->{$pks[0]}"; 
+            }
        } else {
            $params = [];
            foreach ($pks as $pk) {
-                $params[] = "'$pk' => \$model->$pk";
+                if (is_subclass_of($class, 'yii\mongodb\ActiveRecord')) {
+                    $params[] = "'$pk' => (string)\$model->$pk";
+                } else {
+                    $params[] = "'$pk' => \$model->$pk";                   
+                }
            }

            return implode(', ', $params);