Browse Happy API: Replace require() with require_once()

[ronakganatra]

I suggest to use require_once() function in index.php of api.wordpress.org instead of only require()

[ronakganatra]

Changed Require() function to require_once() for improvement.

[SergeyBiryukov]

Hi @ronakganatra, welcome to WordPress Meta Trac!

What problem would that solve?

[ronakganatra]

Replying to SergeyBiryukov:

Hi @ronakganatra, welcome to WordPress Meta Trac!

What problem would that solve?

Hello @SergeyBiryukov Thanks for reviewing it and responding to this ticket below is my feedback :
Currently, any problem is not occurring but I have added this as an improvement. because it adds file every time instead if we use require_once it only include that file once so, efficiency will be improved.

[ocean90]

Thanks for the patch, but there's actually no need for using require_once() here.

I you want to start contributing to meta I suggest to take a look at some of the open tickets: https://meta.trac.wordpress.org/report/1.

[SergeyBiryukov]

Replying to ronakganatra:

I have added this as an improvement. because it adds file every time instead if we use require_once it only include that file once so, efficiency will be improved.

require_once() is actually ​slower than require(), because it has to check if the file has already been included. The performance difference might be negligible, but the function still should not be changed without a good reason.