Project

General

Profile

Misc #14331

change call CGI methods from :: to .

Added by artofhuman (Semyon Pupkov) 6 months ago. Updated 6 months ago.

Status:
Open
Priority:
Normal
Assignee:
-
[ruby-core:84690]

Description

Hi.
I think using :: for call methods for CGI class methods in documentation and core may cause confusion.
I think is better to change this to standard way.
For example CGI::escapeHTML -> CGI.escapeHTML

plz see opened PR on Github https://github.com/ruby/ruby/pull/1749

History

#1 [ruby-core:84691] Updated by shevegen (Robert A. Heiler) 6 months ago

I agree.

I assume that the most likely reason for the ancient style in CGI is that not many use CGI and probably not many maintain it.

The changes mentioned are very simple though. I had a look at:

https://github.com/ruby/ruby/pull/1749/commits/9f684decc2f48c5010095309d7d2b1e2332ea553

and could not see any mistakes so perhaps it could be merged.

Also available in: Atom PDF