php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #56042 PEPr Proposal Update Comment problems
Submitted: 2004-04-25 19:32 UTC Modified: 2004-04-28 01:44 UTC
From: clay at killersoft dot com Assigned: toby (profile)
Status: Closed Package: PECL website (PECL)
PHP Version: Irrelevant OS: Mac OS X 10.2.8
Private report: No CVE-ID: None
View Add Comment Developer Edit
Welcome! If you don't have a Git account, you can't do anything here.
You can add a comment by following this link or if you reported this bug, you can edit this bug over here.
Block user comment
Status: Assign to:
Package:
Bug Type:
Summary:
From: clay at killersoft dot com
New email:
PHP Version: OS:

 

 [2004-04-25 19:32 UTC] clay at killersoft dot com
Description:
------------
Two issues relating to updating a proposal in PEPr:

1. Update comments don't support BBCode. Seems like they could/
should.

2. If the "Send update e-mail to pear-dev" box is not checked, 
changes to the package description are saved, but the 
Update Comments are not saved to the proposal changelog.


Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2004-04-27 00:24 UTC] helgi at trance dot is
assigned it to toby :)
 [2004-04-28 01:07 UTC] helgi at trance dot is
first part of this should be fixed, I'm not sure how to act on the 2) so I emailed toby ... i.e. i'm not sure if you should be able to update changlog if you don't email pear-dev :) toby will email me tomorrow or some about this ;)

Btw. the changelog for you SVN proposal ... is missing 1x [list] :) right after the first [/list]
 [2004-04-28 01:09 UTC] helgi at trance dot is
ohh never mind i said that ... see now the second [list] but this outputs some stranges things on my local server ;) but this has been reported to BBcode package last time i knew :P Sorry! :P
 [2004-04-28 01:24 UTC] helgi at trance dot is
Like I thought ... You shouldn't be able to Update Comment if you don't check the box...

We'll probly add some warning about this!

Thanks :)
 [2004-04-28 01:44 UTC] helgi at trance dot is
This bug has been fixed in CVS.

In case this was a documentation problem, the fix will show up at the
end of next Sunday (CET) on pear.php.net.

In case this was a pear.php.net website problem, the change will show
up on the website in short time.
 
Thank you for the report, and for helping us make PEAR better.

Okey after talking to toby a little, we agreed on removing the checkbox and now if the update comment textarea is filled with some text a email WILL be sent to pear-dev :) also added text to warn users about it ;)

If the text isn't enough or something isn't working then let me know :)
(You probly have to wait a little for these changes since the site has to sync with CVS)
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Tue Apr 23 12:01:31 2024 UTC