Bug 9286 - Mageia bugzilla adds Reproducible & Steps to reproduce on the bottom of all new bugs
Summary: Mageia bugzilla adds Reproducible & Steps to reproduce on the bottom of all n...
Status: RESOLVED FIXED
Alias: None
Product: Infrastructure
Classification: Unclassified
Component: Bugzilla (show other bugs)
Version: unspecified
Hardware: All Linux
Priority: High normal
Target Milestone: ---
Assignee: Frédéric "LpSolit" Buclin
QA Contact:
URL:
Whiteboard:
Keywords:
: 10029 11974 (view as bug list)
Depends on:
Blocks:
 
Reported: 2013-03-07 19:17 CET by claire robinson
Modified: 2016-02-24 01:06 CET (History)
8 users (show)

See Also:
Source RPM:
CVE:
Status comment:


Attachments
Patch to remove useless strings from the initial comment (756 bytes, patch)
2016-02-21 20:39 CET, Frédéric "LpSolit" Buclin
Details | Diff

Description claire robinson 2013-03-07 19:17:02 CET
See bug 9285 for example

That extra text does not show in the comment box when entering the details

Reproducible: 

Steps to Reproduce:
Comment 1 claire robinson 2013-03-07 19:17:26 CET
See above :D
Manuel Hiebel 2013-03-07 20:29:01 CET

CC: (none) => mageia-webteam, sysadmin-bugs
Component: Other => Bugzilla
Version: trunk => unspecified
Product: Websites => Infrastructure

Comment 2 Frédéric "LpSolit" Buclin 2013-03-15 20:42:10 CET
That's due to your customized guided form. The backend code doesn't expect to get these strings and so doesn't strip them.

CC: (none) => LpSolit

Comment 3 claire robinson 2013-03-21 09:44:30 CET
pinging for dmorgan as requested
Comment 4 claire robinson 2013-04-16 17:36:53 CEST
pinging as requested
Comment 5 claire robinson 2013-04-16 17:38:18 CEST
When you get to look at this, can the comments width also be wider?
Comment 6 Frédéric "LpSolit" Buclin 2013-05-10 00:22:21 CEST
*** Bug 10029 has been marked as a duplicate of this bug. ***

CC: (none) => fri

Comment 7 David Walser 2013-12-13 00:28:43 CET
*** Bug 11974 has been marked as a duplicate of this bug. ***

CC: (none) => kristoffer.grundstrom1983

Comment 8 Rémi Verschelde 2015-09-10 16:34:25 CEST
This is a long standing and quite annoying bug. It would be *really* nice if someone who knows his way around bugzilla could have a look at this.

Priority: Normal => High

Comment 9 Morgan Leijström 2015-09-10 17:10:57 CEST
Yes, and wider and auto wrapping comments (both viewing and entering) would be nice.
Comment 10 Frédéric "LpSolit" Buclin 2015-09-10 20:22:45 CEST
(In reply to Morgan Leijström from comment #9)
> Yes, and wider and auto wrapping comments (both viewing and entering) would
> be nice.

Comments are automatically wrapped already. About having a wider textarea, you can manually make it larger if you have a lot to type. Just click on the corner of the text field.


(In reply to Rémi Verschelde from comment #8)
> This is a long standing and quite annoying bug. It would be *really* nice if
> someone who knows his way around bugzilla could have a look at this.

The relevant code is in post_bug.cgi:

my $comment;
my $format = $template->get_format("bug/create/comment",
                                   scalar($cgi->param('format')), "txt");
$template->process($format->{'template'}, $vars, \$comment)
    || ThrowTemplateError($template->error());

You don't need this code anymore as you totally changed the guided form. You should call:

my $comment = $cgi->param('comment');

and parse it to remove the empty sections.
Comment 11 Thomas Backlund 2015-09-10 20:42:17 CEST
@Frederic:

iirc at some point way back you volunteered to maintain Mageia Bugzilla...

is this offer still valid ?

I think you wanted to use git for it back then which we didn't really like as puppet was happier with rpms...

but I think we would now be ok with a git based install much like we do for forum and other stuff

CC: (none) => tmb

Comment 12 Frédéric "LpSolit" Buclin 2015-09-11 17:20:34 CEST
(In reply to Thomas Backlund from comment #11)
> iirc at some point way back you volunteered to maintain Mageia Bugzilla...
> 
> is this offer still valid ?

If you upgrade your server from Mageia 1 to Mageia 5 first, yes. Else it's going to be a pain to maintain Bugzilla. neoclust tried to upgrade Mageia Bugzilla to 5.0 a few weeks ago and had to roll back due to too old Perl modules in Mageia 1.
Comment 13 Thomas Backlund 2015-09-11 20:49:17 CEST
Thanks, and Yes, I meant after we have rolled up infra to mga5
Comment 14 Kristoffer Grundström 2015-09-12 15:31:02 CEST
How far away until this is solved?
Comment 15 Morgan Leijström 2015-09-12 15:54:50 CEST
(In reply to Frédéric Buclin from comment #10)
> (In reply to Morgan Leijström from comment #9)
> > Yes, and wider and auto wrapping comments (both viewing and entering) would
> > be nice.
> 
> Comments are automatically wrapped already. About having a wider textarea,
> you can manually make it larger if you have a lot to type. Just click on the
> corner of the text field.

Yes, that only goes for the edit field where one enters the bug.

As we read comments more often than we enter them, it would be nice if the width of all comment fields would expand when we make the browser window wider, and long lines in them rewrapped.
So when someone enters a few lines of log or code that are long we can widen the window and see them in all theuir glory instead of like now have them wrapped unneccessarily.

(If also the field for entering comment would do this it need not have a handle)
Comment 16 Frédéric "LpSolit" Buclin 2015-11-29 21:22:42 CET
@tmb: is there any ETA for the upgrade of infra to mga5? There are several reported bugs which will be automatically fixed once you upgrade to Bugzilla 5.0.
Comment 17 Samuel Verschelde 2015-12-02 09:52:31 CET
(In reply to Frédéric Buclin from comment #16)
> @tmb: is there any ETA for the upgrade of infra to mga5? There are several
> reported bugs which will be automatically fixed once you upgrade to Bugzilla
> 5.0.

I am myself waiting for bugzilla 5 for work around bugsquad tools, so am interested in the ETA as well.
Comment 18 Thomas Backlund 2016-02-15 17:41:45 CET
@lpsolit:

There is now an empty git for new bugs.mageia.org that you can prepare new bugs.mageia.org setup on:
ssh://git@git.mageia.org/web/bugs

I guess you could set up a "testing" and a "live" branch (and "upstream" ?) or whatever layout you want... 

We'll then wire up git automatic push to new server from git according to the branches you want to use...

We still need to dump the db from bugzilla 4.4 and restore on new db server so you can test on...
Comment 19 Frédéric "LpSolit" Buclin 2016-02-15 18:55:37 CET
@tmb: I guess you need my public SSH key to authenticate? :)
Comment 20 Thomas Backlund 2016-02-15 19:41:59 CET
You should be able to add your key on identity.mageia.org
Comment 21 Frédéric "LpSolit" Buclin 2016-02-15 19:47:53 CET
I see no such field, even in the "add attribute" section at the bottom of the page.
Comment 22 Thomas Backlund 2016-02-15 22:42:51 CET
Oops, sorry, I forgot to promote your account, now try again
Comment 23 Nicolas Lécureuil 2016-02-15 23:43:53 CET
maybe on the new server we can go directly to bugzilla 5 but Frédéric will know better :)

Btw: welcome :)

CC: (none) => mageia

Comment 24 Marja Van Waes 2016-02-16 09:02:44 CET
Welcome, Frédéric :-D

Reassigning this bug to you as welcome gift ;-)

CC: (none) => marja11
Assignee: dmorganec => LpSolit

Comment 25 Frédéric "LpSolit" Buclin 2016-02-16 18:13:19 CET
(In reply to Nicolas Lécureuil from comment #23)
> maybe on the new server we can go directly to bugzilla 5

This would be the plan, yes. Doing the initial work on 4.4 to upgrade right after to 5.0 would be a waste of time, IMO.

That's weird, when I try to git clone ssh://git@git.mageia.org/web/bugs, it asks me for a password, but I use no passphrase. Could this mean that my public SSH key is not recognized by the server?
Comment 26 Frédéric "LpSolit" Buclin 2016-02-21 20:39:25 CET
Created attachment 7479 [details]
Patch to remove useless strings from the initial comment

I'm not allowed to push changes to web/templates/bugzilla, so here is the patch. Please commit.
Comment 27 Mageia Robot 2016-02-21 20:43:53 CET
commit 5a60f9f2f5f070ac7d0f37ed79a3cdb7bc7b91f6
Author: Frédéric Buclin <LpSolit@...>
Date:   Sun Feb 21 21:43:46 2016 +0200

    remove useless strings from the initial comment (mga#9286)
---
 Commit Link:
   http://gitweb.mageia.org/web/templates/bugzilla/commit/?id=5a60f9f2f5f070ac7d0f37ed79a3cdb7bc7b91f6
Comment 28 Frédéric "LpSolit" Buclin 2016-02-21 21:06:10 CET
Reopen if you can still reproduce this issue.

Status: NEW => RESOLVED
Resolution: (none) => FIXED

Comment 29 Frédéric "LpSolit" Buclin 2016-02-24 01:06:27 CET
FYI, I'm done with porting the Mageia extension to Bugzilla 5.0.

http://gitweb.mageia.org/web/bugs/commit/?id=9f4329cf

Note You need to log in before you can comment on or make changes to this bug.