View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0001444||Main CAcert Website||source code||public||2018-10-29 20:18||2018-11-02 18:23|
|Status||needs review & testing||Resolution||open|
|Target Version||2017 Q4||Fixed in Version|
|Summary||0001444: Update PHP <? tags appropriately|
|Description||Go through source code and |
1. change <?= to <? echo
2. change <? to <?php
3. change each() to foreach()
|Additional Information||Part of bug-1260|
|Tags||No tags attached.|
mysql.php seems to be missing from source code for bug-1260. ( should be in includes/mysql.php )
After discussion, I found that this file is "hand-created" on the appropriate server when the code is deployed.
All other required files appear to be present, but they may not be found in a test system because references to them are absolute paths.
||There are hard-coded references to "http://cacert.org," which can probably cause trouble in development and test systems.|
||This code is now available for testing.|
||I found several thousand ( 2500 - 3000 ) instances of required tag changes. Only one instance of each() in the source code that was derived from "release."|
||The ending "?>" tag, at the bottom of PHP source files can be removed.|
I think you shuldn't remove the "?>"tag at the bottom of te PHP source files.
This could cause to assume, some sourcecode could be missing.
Current "best practice" is to omit that tag, because it prevents anything being put into the HTML that is not intended ( extra new lines, extra spaces, etc. ).
On the other hand, I just noted it as something to consider. I did not make this change.
|2018-10-29 20:18||bdmc||New Issue|
|2018-10-29 20:18||bdmc||Assigned To||=> bdmc|
|2018-10-29 22:21||Ted||Relationship added||child of 0001260|
|2018-10-30 04:59||bdmc||Note Added: 0005621|
|2018-10-30 05:00||bdmc||Note Added: 0005622|
|2018-11-02 18:05||bdmc||Note Edited: 0005621||View Revisions|
|2018-11-02 18:07||bdmc||Status||new => solved?|
|2018-11-02 18:07||bdmc||Description Updated||View Revisions|
|2018-11-02 18:07||bdmc||Note Added: 0005641|
|2018-11-02 18:09||bdmc||Note Added: 0005642|
|2018-11-02 18:10||bdmc||Note Added: 0005643|
|2018-11-02 18:11||bdmc||Status||solved? => needs review & testing|
|2018-11-02 18:16||GuKKDevel||Note Added: 0005644|
|2018-11-02 18:23||bdmc||Note Added: 0005645|