Skip to content

Commit

Permalink
Merge pull request #13 from yuhal/master
Browse files Browse the repository at this point in the history
Update Schema.php
  • Loading branch information
BSN4 authored Feb 11, 2022
2 parents a911df5 + 424c397 commit 21c89de
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/Schema.php
Original file line number Diff line number Diff line change
Expand Up @@ -144,7 +144,7 @@ protected function columnsList()
$column = Regex::match('~(["\'])([^"\']+)\1~', $line);
$column = $column->hasMatch() ? $column->group(2) : null;
$types = $this->columnsTypes($column);
$type = Regex::match('/\$.*->(.*)\(/', $line)->group(1);
$type = Regex::match('/\$.*?->(.*?)\(/', $line)->group(1);

return [$line => in_array($type, array_keys($types)) ? $types[$type] : [$column]];
} catch (\Exception $e) {
Expand Down

3 comments on commit 21c89de

@yuhal
Copy link
Contributor

@yuhal yuhal commented on 21c89de Jul 12, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello, author, this question also needs to be released in version 0.2.0?
The current 3. 0 only supports laravel 8 and does not apply to laravel 7 projects.
I hope you can solve it. Thank you very much!

@BSN4
Copy link
Contributor Author

@BSN4 BSN4 commented on 21c89de Jul 12, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@yuhal hello i don't know what you mean it does work with Laravel 7, 8 but rn I only accept PR. i'm not maintaining this repo any more , you can edit composer to support Laravel 9 and make a PR

@yuhal
Copy link
Contributor

@yuhal yuhal commented on 21c89de Jul 13, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, thank you for your prompt reply. I understand now.

Please sign in to comment.