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.

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

fix: return method enum #1469

Closed
wants to merge 1 commit into from
Closed

Conversation

Ichinya
Copy link
Contributor

@Ichinya Ichinya commented Jan 14, 2025

What was changed

MoonShineRequest()->method() выдает строку, при чем в начале выдавал строку маленькими буквами, потом заглавными.

Why?

В коде уже есть enum HttpMethod и логичнее, что должен выдаваться данный enum.

Checklist

  • Issue #
  • Tested
    • [v] Tested manually
    • [-] Tests added
  • [?] Documentation

@lee-to
Copy link
Collaborator

lee-to commented Jan 14, 2025

Ты же пытаешься пофиксить Request и его метод method, муншайн то тут причем? Laravel Request и не должен возвращать наши инамы! Request->method это из вендора ларавел, метод который строкой возвращает http метод, ларавел разработчики еще не доросли до инамов)

@lee-to lee-to closed this Jan 14, 2025
@Ichinya
Copy link
Contributor Author

Ichinya commented Jan 14, 2025

если вызывать request() , то это от Ларавел. Но если MoonShineRequest() - это уже муншайн, и енам тоже в муншайне, разве нет?

@lee-to
Copy link
Collaborator

lee-to commented Jan 14, 2025

если вызывать request() , то это от Ларавел. Но если MoonShineRequest() - это уже муншайн, и енам тоже в муншайне, разве нет?

но MoonShineRequest наследует Request и мы не переопределяем методы родителя, и дело даже не в lsp, нам это не нужно, в этом нет смысла и даже стат анализ не пройдет

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants