Assigned-bugs hackathon!

[denise]

I checked tonight, and we've got 571 open bugs. That's a lot! 146 of them are assigned with no patch, and if we could make some inroads on that number, we'd be able to knock down the total open number a hell of a lot.

So, here's my hackathon challenge! If you've got assigned bugs, let's do whatever we can this week to get that number down. I'll start; I have 8 bugs assigned to me without a patch, and while some of them are waiting on other things to get resolved first, some of them are things I've just been putting off because I don't know exactly how to do them or was running into problems.

I've already yelled for help on what I'm stuck on, so I'll turn to y'all: what can I help you with on stuff you're stuck on? And if not me, who else can help you? A bunch of the existing assigned stuff is really complex, so let's all work together and see how much we can improve things.

Ideally, I'd like to get that "open" number down below 500 by Sunday night, and double ideally, the "all unassigned" number (currently at 386) down under 300.

*salutes you all*

Comments

[cesy]

I need to arrange a time with sophie to attack 1581. She has the code knowledge and I have the design knowledge.

For 581, I need some help figuring out how to add things to the tags box when you click on a link. yvi said I could use Edit Tags, but I'm going to need someone to talk me through it, I think.

And 1877 just needs me to get around to it.

Prod me next time I'm in IRC, maybe? Theoretically I could do one of those tonight.

[denise]

*salute* Yes'm!

[afuna]

For 581, it looks like it requires JavaScript. I... kinda wrote a Greasemonkey script that implemented something similar for LJ. It can't be used here directly, because the bulk of it is irrelevant (also you can use jQuery, which I promise will make life better :)), but I'd be happy to bounce ideas around for your implementation.

[cesy]

Can I use jQuery? I was thinking probably not, because of all the stuff already on the update page. There was some discussion at http://dw-dev-training.dreamwidth.org/10580.html before.

[afuna]

Oh, hm. Yeah, you're right. In that case, the code gets longer and uglier, but not necessarily that much more complicated (e.g., lots more explicitly setting the values, instead of convenience methods). So scratch what I said about jQuery then. Oops.

[cesy]

Will roughly the right Javascript be on the Edit Tags page, or will I need to write new stuff?

[afuna]

Hmmm. I'm not sure, TBH! The main difference between edit tags and the tag cloud is that edit tags uses the select form element. So I think you could base your approach on it, but if it's using an onchange handler , you'll need to make sure to be using an onclick.

(I haven't looked closely at the implementation)

[yvi]

Um, yeah, I kind of failed to see that you have to use JavaScript there, which I am really rubbish at.

[cesy]

Notes on bug 581, if anyone's free to help:

I have scalar %$tags which tells me the number of items in $tags, so I can construct appropriate if statements, and values %$tags gets the actual values. How can I get the values of only the first 10 items? Is there syntax to say "get the nth item from this array"? The relevant line is currently:
$out .= LJ::ehtml(join ', ', sort map { $_->{name} } values %$tags);
which gets all the items in the array.

[yvi]

Is there a reason you are not using an array? Because with that, you could just use split to get the first ten items. I am not sure how it works with hashes.

[yvi]

*facepalm* a) I meant splice

b) Or you can probably use splice on values %$tags.

my @10first = splice ( values %$tags , 0 , 10 ) or something like that.

[pauamma]

Something like:

$out .= LJ::ehtml(join ', ', sort { $a cmp $b } map { $_->{name} } (values %$tags)[0..9]);

if you want the first 10 without sorting, or:

$out .= LJ::ehtml(join ', ', (sort { $a cmp $b } map { $_->{name} } values %$tags)[0..9]);

if you want the first 10 sorted by the name field.

(Also, added the missing comparison function to your sort, assuming you want string ordering. Adjust as needed.)

[cesy]

Thanks, that works :)