Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Internal: Update code intelligence class parsing to evaluate Alpine classes #1619

Merged
merged 10 commits into from
Mar 18, 2024
Merged

Internal: Update code intelligence class parsing to evaluate Alpine classes #1619

Changes from all commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
bf8ec1d
Exclude dynamic Alpine classes from class parser
caendesilva Mar 18, 2024
70f7462
Refine regex pattern
caendesilva Mar 18, 2024
80462ab
Revert "Refine regex pattern"
caendesilva Mar 18, 2024
2868770
Revert "Exclude dynamic Alpine classes from class parser"
caendesilva Mar 18, 2024
eff1c31
Draft new logic abstract
caendesilva Mar 18, 2024
53345ac
Implement logic using ChatGPT
caendesilva Mar 18, 2024
dee1379
Merge branch 'master' into update-internal-monorepo-tools
caendesilva Mar 18, 2024
438ce63
Strongly type closure types
caendesilva Mar 18, 2024
1d62edc
Convert concatenation to 'sprintf()' call
caendesilva Mar 18, 2024
f6b0856
Add some spacing and clean up method formatting
caendesilva Mar 18, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 16 additions & 0 deletions monorepo/CodeIntelligence/CodeIntelligence.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -496,8 +496,24 @@ protected function findBladeElementClasses(): array
$classes = [];

foreach ($this->bladeFiles as $contents) {
// Extract static classes from dynamic Alpine.js classes containing boolean logic
$contents = preg_replace_callback('/:class=(["\'])(.*?)\1/', function (array $matches): string {
$staticClasses = '';
$dynamicClasses = $matches[2];

// Parse the dynamic classes to separate static classes
preg_match_all('/\'\s*([^\'\s]*)\s*\'/', $dynamicClasses, $staticMatches);
if (! empty($staticMatches[1])) {
$staticClasses = sprintf('class="%s"', implode(' ', $staticMatches[1]));
}

return $staticClasses;
}, $contents);

$matches = [];

preg_match_all('/class="([^"]+)"/', $contents, $matches);

foreach ($matches[1] as $match) {
$match = explode(' ', $match);
foreach ($match as $class) {
Expand Down
Loading