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

Jump to bottom

1. Replace usage of global with dependency injection #72

Open
agilare opened this issue Sep 6, 2024 · 0 comments
Open

1. Replace usage of global with dependency injection #72

agilare opened this issue Sep 6, 2024 · 0 comments
Assignees
@agilare
Labels
high priority refactoring

Comments

@agilare
Copy link
Owner

agilare commented Sep 6, 2024

Guidelines

  • process : replace by class one by one, not by global var
  • try to convert statics methods to non static and then instanciate its class

global

All are important services or configurations defined in app/

  • config.php
  • bootstrap.php

and used in libraries/ as

  • $connector
  • $authorization
  • $logger
  • $glo...
  • $rep...

$GLOBALS

Not used in the application
@agilare agilare changed the title Replace usage of global with dependency injection 1. Replace usage of global with dependency injection Sep 9, 2024
@agilare agilare moved this to Todo in Modernization Sep 9, 2024
@agilare agilare self-assigned this Sep 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@agilare agilare

Labels
high priority refactoring
Projects
Modernization
Status: Todo
Milestone
No milestone
Development

No branches or pull requests
1 participant
@agilare