views_handler_argument_node_type should extend views_handler_argument_string and not just views_handler_argument. This came about from a support request on IRC. Also, after speaking to dawehner, thought this may only be for historical reasons.

Patch attached, have run all views tests, just to make sure, and all passes.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dawehner’s picture

Status: Needs review » Reviewed & tested by the community

This looks fine from my perspective.

dawehner’s picture

Status: Reviewed & tested by the community » Fixed

Committed now due to a request in irc :)

Thanks!

damiankloip’s picture

ha, I think the request came from the same person too :)

jnor’s picture

Thanks guys! Thumbs up

Status: Fixed » Closed (fixed)

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