As the title says.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kotnik’s picture

Status: Active » Needs review
FileSize
1.1 KB

Fix attached.

Lars Toomre’s picture

Status: Needs review » Reviewed & tested by the community

Change makes sense.

jhodgdon’s picture

Assigned: Unassigned » jhodgdon

Looks fine, thanks! Normally I would say I wouldn't commit a patch that hasn't been run through the test bot (you should not have put the -do-not-test suffix on the file name), but since it's a one-line documentation fix, I'll make an exception in this case (pretty clear it does not introduce a PHP problem). I'll get it committed shortly.

Lars Toomre’s picture

@jhodgdon -- Slightly off-topic -- I am unsure of our current policy regarding classes referenced in a docblock like this one. Are we supposed to start them with a fully qualified '\' or leave them as is?

jhodgdon’s picture

Lars: http://drupal.org/node/1354#namespaces -- that issue is going to require some patching but not until after feature freeze (Dec 1).

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Nice catch!

Committed and pushed to 8.x. Sorry, Jennifer, I was looking for something easy to do at 2:30am. ;)

jhodgdon’s picture

Assigned: jhodgdon » Unassigned

Um. So first you apologize when you assign me issues to commit, and now you apologize for committing a patch that I would have had to commit? What's up with that? :)

Which is to say: no apology necessary! I have *plenty* of backlog, since I haven't done any commits for about 6 days now. The important thing is to get these patches committed... speaking of which...

webchick’s picture

Well, I am Canadian, you know. :D

Status: Fixed » Closed (fixed)

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