standardize multiline ternaries (#55056)

if either the truthy or falsy expression is on a newline, make them both on a new line, and indent exactly once.

I don't think we have to go so far as to say ALL ternaries need to be multi-line, but I would say if one expression is, they both should be.

will make for easy readability and better diffs.

unable to found an automation rule for this yet, but will keep looking.

Author: browner12

src/Illuminate/Broadcasting/BroadcastEvent.php

@@ -75,7 +75,8 @@ public function __construct($event)
     public function handle(BroadcastingFactory $manager)
     {
         $name = method_exists($this->event, 'broadcastAs')
-            ? $this->event->broadcastAs() : get_class($this->event);
+            ? $this->event->broadcastAs()
+            : get_class($this->event);
 
         $channels = Arr::wrap($this->event->broadcastOn());
 

src/Illuminate/Database/Eloquent/Concerns/HasRelationships.php

@@ -821,8 +821,9 @@ public function joiningTable($related, $instance = null)
         // sorted alphabetically and concatenated with an underscore, so we can
         // just sort the models and join them together to get the table name.
         $segments = [
-            $instance ? $instance->joiningTableSegment()
-                      : Str::snake(class_basename($related)),
+            $instance
+                ? $instance->joiningTableSegment()
+                : Str::snake(class_basename($related)),
             $this->joiningTableSegment(),
         ];
 

src/Illuminate/Database/Eloquent/Model.php

@@ -1095,7 +1095,8 @@ public function push()
             // us to recurse into all of these nested relations for the model instance.
             foreach ($this->relations as $models) {
                 $models = $models instanceof Collection
-                    ? $models->all() : [$models];
+                    ? $models->all()
+                    : [$models];
 
                 foreach (array_filter($models) as $model) {
                     if (! $model->push()) {

src/Illuminate/Database/Eloquent/Relations/MorphToMany.php

@@ -157,8 +157,9 @@ public function newPivot(array $attributes = [], $exists = false)
 
         $attributes = array_merge([$this->morphType => $this->morphClass], $attributes);
 
-        $pivot = $using ? $using::fromRawAttributes($this->parent, $attributes, $this->table, $exists)
-                        : MorphPivot::fromAttributes($this->parent, $attributes, $this->table, $exists);
+        $pivot = $using
+            ? $using::fromRawAttributes($this->parent, $attributes, $this->table, $exists)
+            : MorphPivot::fromAttributes($this->parent, $attributes, $this->table, $exists);
 
         $pivot->setPivotKeys($this->foreignPivotKey, $this->relatedPivotKey)
             ->setRelatedModel($this->related)

src/Illuminate/Database/Query/Builder.php

@@ -3306,7 +3306,8 @@ protected function withoutSelectAliases(array $columns)
     {
         return array_map(function ($column) {
             return is_string($column) && ($aliasPosition = stripos($column, ' as ')) !== false
-                ? substr($column, 0, $aliasPosition) : $column;
+                ? substr($column, 0, $aliasPosition)
+                : $column;
         }, $columns);
     }
 
@@ -3634,7 +3635,8 @@ public function numericAggregate($function, $columns = ['*'])
         // cast it to one. When it does we will cast it to a float since it needs to be
         // cast to the expected data type for the developers out of pure convenience.
         return ! str_contains((string) $result, '.')
-            ? (int) $result : (float) $result;
+            ? (int) $result
+            : (float) $result;
     }
 
     /**

src/Illuminate/Database/Schema/BlueprintState.php

@@ -77,9 +77,11 @@ public function __construct(Blueprint $blueprint, Connection $connection)
             'collation' => $column['collation'],
             'comment' => $column['comment'],
             'virtualAs' => ! is_null($column['generation']) && $column['generation']['type'] === 'virtual'
-                ? $column['generation']['expression'] : null,
+                ? $column['generation']['expression']
+                : null,
             'storedAs' => ! is_null($column['generation']) && $column['generation']['type'] === 'stored'
-                ? $column['generation']['expression'] : null,
+                ? $column['generation']['expression']
+                : null,
         ]))->all();
 
         [$primary, $indexes] = (new Collection($schema->getIndexes($table)))->map(fn ($index) => new IndexDefinition([

src/Illuminate/Database/Schema/Grammars/MySqlGrammar.php

@@ -381,9 +381,11 @@ protected function compileLegacyRenameColumn(Blueprint $blueprint, Fluent $comma
             'collation' => $column['collation'],
             'comment' => $column['comment'],
             'virtualAs' => ! is_null($column['generation']) && $column['generation']['type'] === 'virtual'
-                ? $column['generation']['expression'] : null,
+                ? $column['generation']['expression']
+                : null,
             'storedAs' => ! is_null($column['generation']) && $column['generation']['type'] === 'stored'
-                ? $column['generation']['expression'] : null,
+                ? $column['generation']['expression']
+                : null,
         ]));
 
         return sprintf('alter table %s change %s %s %s',

src/Illuminate/Events/Dispatcher.php

@@ -344,7 +344,8 @@ protected function shouldBroadcast(array $payload)
     protected function broadcastWhen($event)
     {
         return method_exists($event, 'broadcastWhen')
-            ? $event->broadcastWhen() : true;
+            ? $event->broadcastWhen()
+            : true;
     }
 
     /**

src/Illuminate/Foundation/Testing/DatabaseTruncation.php

@@ -141,7 +141,8 @@ protected function tableExistsIn(array $table, array $tables): bool
     protected function connectionsToTruncate(): array
     {
         return property_exists($this, 'connectionsToTruncate')
-            ? $this->connectionsToTruncate : [null];
+            ? $this->connectionsToTruncate
+            : [null];
     }
 
     /**

src/Illuminate/Routing/Route.php

@@ -792,7 +792,8 @@ public function domain($domain = null)
     public function getDomain()
     {
         return isset($this->action['domain'])
-            ? str_replace(['http://', 'https://'], '', $this->action['domain']) : null;
+            ? str_replace(['http://', 'https://'], '', $this->action['domain'])
+            : null;
     }
 
     /**