Req #53792 [Asn->Csd]: windows.php.net download page, archive link (svn diff and file appended)
| From: | kalle@php.net | Date: | Thu, 20 Jan 2011 18:13:18 +0000 |
| Subject: | Req #53792 [Asn->Csd]: windows.php.net download page, archive link (svn diff and file appended) | ||
| References: | 1 | Groups: | php.webmaster |
| Request: | Send a blank email to php-webmaster+get-10461@lists.php.net to get a copy of this message | ||
Edit report at http://bugs.php.net/bug.php?id=53792&edit=1
ID: 53792
Updated by: kalle@php.net
Reported by: harlequin2 at gmx dot de
Summary: windows.php.net download page, archive link (svn
diff and file appended)
-Status: Assigned
+Status: Closed
Type: Feature/Change Request
Package: Website problem
PHP Version: Irrelevant
Assigned To: kalle
Block user comment: N
Private report: N
New Comment:
This bug has been fixed in SVN. Since the websites are not directly
updated from the SVN server, the fix might need some time to spread
across the globe to all mirror sites, including PHP.net itself.
Thank you for the report, and for helping us make PHP.net better.
I added the archive link, however I took abit different approach than
your diff and added it as a sidebar widget/box instead which I think is
better suited for such meta information.
Previous Comments:
------------------------------------------------------------------------
[2011-01-20 19:12:27] kalle@php.net
Automatic comment from SVN on behalf of kalle
Revision: http://svn.php.net/viewvc/?view=revision&revision=307632
Log: Implemented FR #53792 (winweb browse archive link)
------------------------------------------------------------------------
[2011-01-19 21:11:46] bjori@php.net
I don't have windowsweb karma.. Kalle; could you check this out?
------------------------------------------------------------------------
[2011-01-19 17:21:59] harlequin2 at gmx dot de
Description:
------------
As requested on the internals mailing list, here's the modified download
page for windows.php.net. A new box with a link to the currently
selected version's release archive page (releases/qa/snapshots) will be
appended to the bottom of the list. I also started to add some notes to
the code (i.e. closing bracket comments), but the page should definitely
be cleaned up a lot more to make future changes easier.
Regards, Sascha Meyer
--- SVN diff ----
# This patch file was generated by NetBeans IDE
# It uses platform neutral UTF-8 encoding and \n newlines.
--- Ausgangspunkt (BASE)
+++ Lokal geändert (ausgehend von LOCAL)
@@ -21,6 +21,7 @@
$baseurl = '/' . $dir_to_parse . '/';
$versions = generate_listing($dir_to_parse, $snaps);
+
$major_order = array('5.3', '5.2', '6.0');
$minor_order = array(
'5.2' => array(
@@ -69,7 +70,7 @@
<div class="adv-block"><!-- .adv-block -->
<span
class="corners-top"><span></span></span>
- <div class="info entry"><!-- .info -->
+ <div class="info entry"><!-- .info
(release dropdown box) -->
<h2 name="top" id="top"><?php echo
$title_page; ?></h2>
<select
onchange="window.location.hash=this.options[this.selectedIndex].value">
@@ -103,7 +104,7 @@
</div><span
class="corners-bottom"><span></span></span></div>
<?php
continue;
- }
+ } // END if (!isset($versions[$major])) ...
?>
<p><a href="<?php echo $baseurl .
$versions[$major]['source']['path'];
?>">Download source code</a> [<?php echo
$versions[$major]['source']['size']; ?>]</p>
@@ -117,10 +118,8 @@
<?php
foreach ($minor_order[$major] as $minor) {
+ if (isset($versions[$major][$minor])) {
?>
-
-<?php if (isset($versions[$major][$minor])) {
-?>
<div class="innerbox">
<span class="corners-top"><span></span></span>
@@ -166,18 +165,31 @@
</div><!-- innerbox -->
<?php
- }
+ } // END if (isset($versions[$major][$minor]))
+ } // END foreach ($minor_order[$major] ...
?>
- <?php
- }
- ?>
</div>
<span class="corners-bottom"><span></span></span>
</div> <!-- block -->
<?php
- }
+ } // END foreach ($major_order ...
?>
+ <div class="block">
+ <span class="corners-top"></span>
+
+ <div class="info entry">
+ <h3 id="php-archive" name="php-archive" class="summary
entry-title">Browse archive</h3>
+ <div class="innerbox">
+ <span class="corners-top"><span></span></span>
+ <a href="<?php print $baseurl = '/' . $dir_to_parse . '/';
?>"><?php print $title_page;?></a>
+ <span class="corners-bottom"></span>
+ </div>
+ <span class="corners-bottom"></span>
+ </div>
+ </div>
+
+
<p class="t-center">
</p>
</div> <!-- .content -->
------------------------------------------------------------------------
--
Edit this bug report at http://bugs.php.net/bug.php?id=53792&edit=1