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.

CommentFileSizeAuthor
views_node_type_user_string_handler.patch543 bytesdamiankloip

Comments

dawehner’s picture

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

This looks fine from my perspective.

dawehner’s picture

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

Committed now due to a request in irc :)

Thanks!

damiankloip’s picture

damiankloip commented

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

jnor’s picture

jnor commented

Thanks guys! Thumbs up

Status: Fixed » Closed (fixed)

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