[PEAR-BUG] Req #9127 [NEW]: extending make_ticket_links() matched words

[l dot alberton at quipo dot it]

From: l dot alberton at quipo dot it
Operating system:
Package version:
PHP version: 5.1.0
Package: Web Site
Bug Type: Feature/Change Request
Bug description: extending make_ticket_links() matched words

Description:
------------
Attached, a patch to pearweb/include/pear-format-html.php
containing a change to the regexp in make_ticket_links()
to match these cases:

bugfix XXX
bug XXX
req XXX
request XXX

At the moment, only these are supported:

bug XXX
request XXX

Test script:
---------------
--- /tmp/pear-format-html.1.191.php Sat Oct 21 12:18:11 2006
+++ /CVS_repository/pearweb/include/pear-format-html.php Sat Oct 21
12:18:02 2006
@@ -1142,10 +1142,10 @@
*/
function make_ticket_links($text)
{
- $text = preg_replace('/(?<=php)\s*(bug|request)\s+#?([0-9]+)/i',
+ $text =
preg_replace('/(?<=php)\s*(bug(?:fix)?|req(?:uest)?)\s+#?([0-9]+)/i',
' <a href="http://bugs.php.net/\\2">\\1
\\2</a>',
$text);
- $text = preg_replace('/(?<![>a-z])(bug|request)\s+#?([0-9]+)/i',
+ $text =
preg_replace('/(?<![>a-z])(bug(?:fix)?|req(?:uest)?)\s+#?([0-9]+)/i',
'<a href="/bugs/\\2">\\0</a>', $text);
return $text;
}

[m***@php.net]

Edit report at http://pear.php.net/bugs/bug.php?id=9127&edit=1

ID: 9127
Updated by: ***@php.net
Reported By: l dot alberton at quipo dot it
-Status: Open
+Status: Assigned
Type: Feature/Change Request
Package: Web Site
PHP Version: 5.1.0
-Assigned To:
+Assigned To: mj
New Comment:

I'll make sure that this gets in.


Previous Comments:
------------------------------------------------------------------------

[2006-10-21 18:14:25] l dot alberton at quipo dot it

Description:
------------
Attached, a patch to pearweb/include/pear-format-html.php
containing a change to the regexp in make_ticket_links()
to match these cases:

bugfix XXX
bug XXX
req XXX
request XXX

At the moment, only these are supported:

bug XXX
request XXX

Test script:
---------------
--- /tmp/pear-format-html.1.191.php Sat Oct 21 12:18:11 2006
+++ /CVS_repository/pearweb/include/pear-format-html.php Sat Oct 21
12:18:02 2006
@@ -1142,10 +1142,10 @@
*/
function make_ticket_links($text)
{
- $text = preg_replace('/(?<=php)\s*(bug|request)\s+#?([0-9]+)/i',
+ $text =
preg_replace('/(?<=php)\s*(bug(?:fix)?|req(?:uest)?)\s+#?([0-9]+)/i',
' <a href="http://bugs.php.net/\\2">\\1
\\2</a>',
$text);
- $text = preg_replace('/(?<![>a-z])(bug|request)\s+#?([0-9]+)/i',
+ $text =
preg_replace('/(?<![>a-z])(bug(?:fix)?|req(?:uest)?)\s+#?([0-9]+)/i',
'<a href="/bugs/\\2">\\0</a>', $text);
return $text;
}

------------------------------------------------------------------------

[m***@php.net]

Edit report at http://pear.php.net/bugs/bug.php?id=9127&edit=1

ID: 9127
Updated by: ***@php.net
Reported By: l dot alberton at quipo dot it
-Status: Assigned
+Status: Closed
Type: Feature/Change Request
Package: Web Site
PHP Version: 5.1.0
Assigned To: mj
New Comment:

This bug has been fixed in CVS.

If this was a documentation problem, the fix will appear on
pear.php.net by the end of next Sunday (CET).

If this was a problem with the pear.php.net website, the change should
be live shortly.

Otherwise, the fix will appear in the package's next release.

Thank you for the report and for helping us make PEAR better.




Previous Comments:
------------------------------------------------------------------------

[2007-02-24 10:05:03] mj at php dot net

I'll make sure that this gets in.

------------------------------------------------------------------------

[2006-10-21 18:14:25] l dot alberton at quipo dot it

Description:
------------
Attached, a patch to pearweb/include/pear-format-html.php
containing a change to the regexp in make_ticket_links()
to match these cases:

bugfix XXX
bug XXX
req XXX
request XXX

At the moment, only these are supported:

bug XXX
request XXX

Test script:
---------------
--- /tmp/pear-format-html.1.191.php Sat Oct 21 12:18:11 2006
+++ /CVS_repository/pearweb/include/pear-format-html.php Sat Oct 21
12:18:02 2006
@@ -1142,10 +1142,10 @@
*/
function make_ticket_links($text)
{
- $text = preg_replace('/(?<=php)\s*(bug|request)\s+#?([0-9]+)/i',
+ $text =
preg_replace('/(?<=php)\s*(bug(?:fix)?|req(?:uest)?)\s+#?([0-9]+)/i',
' <a href="http://bugs.php.net/\\2">\\1
\\2</a>',
$text);
- $text = preg_replace('/(?<![>a-z])(bug|request)\s+#?([0-9]+)/i',
+ $text =
preg_replace('/(?<![>a-z])(bug(?:fix)?|req(?:uest)?)\s+#?([0-9]+)/i',
'<a href="/bugs/\\2">\\0</a>', $text);
return $text;
}

------------------------------------------------------------------------

[c***@php.net]

Edit report at http://pear.php.net/bugs/bug.php?id=9127&edit=1

ID: 9127
Updated by: ***@php.net
Reported By: l dot alberton at quipo dot it
Status: Closed
Type: Feature/Change Request
Package: Web Site
PHP Version: 5.1.0
Assigned To: mj
New Comment:

testing request 9127, bugfix 9127, bug 9127, req 9127


Previous Comments:
------------------------------------------------------------------------

[2007-02-24 10:38:34] mj at php dot net

This bug has been fixed in CVS.

If this was a documentation problem, the fix will appear on
pear.php.net by the end of next Sunday (CET).

If this was a problem with the pear.php.net website, the change should
be live shortly.

Otherwise, the fix will appear in the package's next release.

Thank you for the report and for helping us make PEAR better.

------------------------------------------------------------------------

[2007-02-24 10:05:03] mj at php dot net

I'll make sure that this gets in.

------------------------------------------------------------------------

[2006-10-21 18:14:25] l dot alberton at quipo dot it

Description:
------------
Attached, a patch to pearweb/include/pear-format-html.php
containing a change to the regexp in make_ticket_links()
to match these cases:

bugfix XXX
bug XXX
req XXX
request XXX

At the moment, only these are supported:

bug XXX
request XXX

Test script:
---------------
--- /tmp/pear-format-html.1.191.php Sat Oct 21 12:18:11 2006
+++ /CVS_repository/pearweb/include/pear-format-html.php Sat Oct 21
12:18:02 2006
@@ -1142,10 +1142,10 @@
*/
function make_ticket_links($text)
{
- $text = preg_replace('/(?<=php)\s*(bug|request)\s+#?([0-9]+)/i',
+ $text =
preg_replace('/(?<=php)\s*(bug(?:fix)?|req(?:uest)?)\s+#?([0-9]+)/i',
' <a href="http://bugs.php.net/\\2">\\1
\\2</a>',
$text);
- $text = preg_replace('/(?<![>a-z])(bug|request)\s+#?([0-9]+)/i',
+ $text =
preg_replace('/(?<![>a-z])(bug(?:fix)?|req(?:uest)?)\s+#?([0-9]+)/i',
'<a href="/bugs/\\2">\\0</a>', $text);
return $text;
}

------------------------------------------------------------------------