We need a rule to handle PSR-0 compatible class loader in core.

Comments

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Issue summary: View changes
donquixote’s picture

donquixote CreditAttribution: donquixote commented

What is needed here?
As I see it:
1. Transform the patch in #2083547: PSR-4: Putting it all together to use src instead of lib.
2. Commit #2083547: PSR-4: Putting it all together (without moving the files)
3. Close #2083547: PSR-4: Putting it all together.
4. Continue in this issue.
5. Run the switch-psr4.sh script, and upload the diff as a patch, in this issue.
6. Re-roll until someone commits it.

Correct?

EDIT:
Sorry, totally missed which project this issue is for. So the above totally misses the point.
This said: The core script does work for modules! You just need to specify a folder as a commandline argument.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented

Good point. :) I might be able to just pull that in!

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Project: Drupal 7 to 8 Module Upgrader [dead] » Drupal 7 to 8/9/10 Module Upgrader

Moving to new project.

  • Commit 6d3b507 on master, moveallthethings, psr, console by webchick: 
    Issue #2194001: Adding a starter sniff for PSR-*
  • Commit eaa638d on master, moveallthethings, psr, console by webchick:
    Issue #2194001: Add some errors around PSR-0/PSR-4.
  • Commit 11818e4 on master, moveallthethings, psr, console by webchick:
    Issue #2194001: Better way of checking for more than one class in a file...

  • webchick committed 6d3b507 on refactor 
    Issue #2194001: Adding a starter sniff for PSR-*
  • webchick committed eaa638d on refactor
    Issue #2194001: Add some errors around PSR-0/PSR-4.
  • webchick committed 11818e4 on refactor
    Issue #2194001: Better way of checking for more than one class in a file...
phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
CreditAttribution: phenaproxima commented
Status: Active » Closed (won't fix)

I'm pretty sure this no longer applies to DMU, and there's a separate issue for PSR-4 conversion. Closing.

  • webchick committed 6d3b507 on 8.x-1.x 
    Issue #2194001: Adding a starter sniff for PSR-*
  • webchick committed eaa638d on 8.x-1.x
    Issue #2194001: Add some errors around PSR-0/PSR-4.
  • webchick committed 11818e4 on 8.x-1.x
    Issue #2194001: Better way of checking for more than one class in a file...

  • webchick committed 6d3b507 on injection 
    Issue #2194001: Adding a starter sniff for PSR-*
  • webchick committed eaa638d on injection
    Issue #2194001: Add some errors around PSR-0/PSR-4.
  • webchick committed 11818e4 on injection
    Issue #2194001: Better way of checking for more than one class in a file...