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
Rename deleteHeader()
to unsetHttpHeader()
#22
Comments
Actually, "delete" isn't really the right term here. I'd say: Either unsetHttpHeader(), or (to be consistent with |
A PR to fix this could be implemented without breaking BC. Just add an alias for the "wrong" function name and mark it deprecated. |
ThomasLandauer
added a commit
to ThomasLandauer/module-rest
that referenced
this issue
Oct 14, 2020
Open
If you merge this, I'll do the same in lib-innerbrowser. |
Naktibalda
changed the title
Rename
Rename Oct 24, 2020
deleteHeader()
to deleteHttpHeader()
deleteHeader()
to unsetHttpHeader()
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
To be consistent with all other header functions:
haveHttpHeader()
(to which it is the counterpart),seeHttpHeader()
,seeHttpHeaderOnce()
, etc.Also in lib-innerbrowser
The text was updated successfully, but these errors were encountered: