Restore Views UID validatory, use $argument rather than arg(1) [#1707098]

Comment	File	Size	Author
#1	twitter-1707098-use_argument_not_arg-7.x-3.x.patch	1.73 KB	xurizaemon
#1	twitter-1707098-use_argument_not_arg-6.x-3.x.patch	1.52 KB	xurizaemon

Comment #1

he/him

Ōtepoti, Aotearoa 🏝

commented 31 July 2012 at 22:05

Version:	6.x-3.x-dev	» 7.x-3.x-dev
Status:	Active	» Needs review

Status	File	Size
new	twitter-1707098-use_argument_not_arg-6.x-3.x.patch	1.52 KB
new	twitter-1707098-use_argument_not_arg-7.x-3.x.patch	1.73 KB

Was working on 6.x-3.x when I spotted this so 7.x-3.x is untested currently but should be identical.

Log in or register to post comments

Comment #2

xurizaemon

he/him

Ōtepoti, Aotearoa 🏝

commented 31 July 2012 at 22:58

Status:

Needs review

» Needs work

D7 uses arrays for user_load() so the second patch above won't work.

Log in or register to post comments

Comment #3

dddave commented 16 May 2013 at 17:52

Version:

7.x-3.x-dev

» 7.x-6.x-dev

still relevant?

Log in or register to post comments

Comment #4

Yorgg commented 27 August 2013 at 16:43

Yes.
I need the contextual filter UID to be %

Log in or register to post comments

Comment #5

xurizaemon

he/him

Ōtepoti, Aotearoa 🏝

commented 27 August 2013 at 21:36

Title:

Restore Views UID validator, use $argument rather than arg(1)

» Restore Views UID validatory, use $argument rather than arg(1)

The code this issue was originally reported against is not in 7.x-6.x, it's been removed entirely in 13fa2451 as part of
#1847406: Move account listing to the module settings, the concept being that "removing the relationship between a Drupal user and the Twitter accounts would simplify the module and the user experience".

But the uid mapping was restored, because it's essential to some uses of the module (ie a forum with per-user Twitter integration).

Decision AFAIK was not to remove the user relationship entirely but to make it optional. Looks like the views filter was forgotten in that, and will need to be restored to be fixed :)