I think we need to create simple unit test to create menu item with title user/[user-name] and link user/[user-uid].

CommentFileSizeAuthor
#1 1068338-menu_token-test.patch1.93 KBygerasimov
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ygerasimov’s picture

ygerasimov CreditAttribution: ygerasimov commented
Status: Active » Needs review
FileSize
1068338-menu_token-test.patch1.93 KB

patch attached

dealancer’s picture

dealancer CreditAttribution: dealancer commented
Status: Needs review » Reviewed & tested by the community

Looks good for me. Could be committed. Want me to do that?

DevElCuy’s picture

DevElCuy CreditAttribution: DevElCuy commented

You don't need to RTBC if you are a maintainer and consider a patch fixed, but is preferable that maintainers commit RTBC'ed patches (it depends on community interest).

dealancer’s picture

dealancer CreditAttribution: dealancer commented
Status: Reviewed & tested by the community » Fixed

So I have committed this patch in order to allow us to work on other issues.

DevElCuy’s picture

DevElCuy CreditAttribution: DevElCuy commented
Status: Fixed » Closed (fixed)

Great stuff!