Was adding an argument, and saw the following string (in the taxonomy argument plugin):
"Load default argument from node page, thats good for related taxonomy blocks."

Attaching a one-line patch that fixes this.
Might be even nicer to say "Load default argument from node page. Good for related taxonomy blocks.", but that's for you to decide.

The issue is silly, but hey, a little polish never hurt anyone ;)

CommentFileSizeAuthor
#1 alternative.patch561 bytesbojanz
fix.patch568 bytesbojanz

Comments

bojanz’s picture

bojanz commented
StatusFileSize
new alternative.patch561 bytes

Attaching the alternative patch too (the second suggested string).

dawehner’s picture

dawehner
Primary language German
commented
Status: Needs review » Reviewed & tested by the community

Looks fine.

This patch should be first commited to 2.x and 3.x

bojanz’s picture

bojanz commented

Which one, the first or the second one?
I can roll it for other branches...

dawehner’s picture

dawehner
Primary language German
commented

Ah. I just read the second one. I'm not an expert in english grammar, but the second looked nice for me.

bojanz’s picture

bojanz commented

Just tested, the patch (alternative.patch) also applies to the DRUPAL--6-3 and DRUPAL--6-2 branches.

Awesome!

dawehner’s picture

dawehner
Primary language German
commented
Version: 7.x-3.x-dev » 6.x-3.x-dev

.

merlinofchaos’s picture

merlinofchaos commented
Status: Reviewed & tested by the community » Fixed

Please try to roll patches from the 'views' root directory, it's much easier to apply them that way.

Please don't change the quoting. ' is the preferred quoting to use on t() unless there is a ' in the sentence.

OTHER than that. Committed to all branches.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.