Userreference, impossible to filter allowed values by blocked users [#416134]

Comment	File	Size	Author
#9	userreference-416134-9.patch	3.83 KB	markus_petrux

Comment #1

petrescs CreditAttribution: petrescs commented 5 May 2009 at 14:42

subscribing - checking Blocked status box is not saved, only the default marked Active checkbox is preserved

Log in or register to post comments

Comment #2

aharown07 CreditAttribution: aharown07 commented 15 June 2009 at 19:05

Also having this problem. We would like very much to be able to reference blocked users when we write our user histories. Anybody got patch ideas? Beyond my skills unfortunately.

Log in or register to post comments

Comment #3

aharown07 CreditAttribution: aharown07 commented 15 June 2009 at 19:30

Version:

6.x-2.2

» 6.x-2.3

Workaround: if you have Views, a View that includes both blocked and unblocked does work as the source for the user ref field... just make the View then choose it under Advanced.
Still a bug though since there's a checkbox there that doesn't work.

Log in or register to post comments

Comment #4

hefox CreditAttribution: hefox commented 15 June 2009 at 20:16

An easy but wonky way:
1) make a a define('BLOCKED_USER_VALUE',2) or maybe 'blocked'.
2) in the form have BLOCKED_USER_VALUE instead of 0 as the key
3) Replace BLOCKED_USER_VALUE with 0 for DB call.
(Don't have the code in front of me so no line numbers. but using above step 2 was probably line 75 and step 3 was probably line 792).

Log in or register to post comments

Comment #5

ize CreditAttribution: ize commented 5 August 2009 at 10:38

Version:

6.x-2.3

» 6.x-2.5

subscribe

Log in or register to post comments

Comment #6

markus_petrux CreditAttribution: markus_petrux commented 8 August 2009 at 17:16

Status:

Needs review

» Active

Probably the user status option would have to be converted into a 'radios' element with the following options:

array(
  '' => t('All users'),
  1 => t('Active users'),
  0 => t('Blocked users'),
)

To do so, we need to change a little the code in userreference.module, but also implement a new hook_update_N() function to migrate the field settings to the new format (from array() to integer).

If Active or Blocked has been selected can be checked using is_numeric().

Log in or register to post comments

Comment #7

DarrellDuane CreditAttribution: DarrellDuane commented 6 August 2009 at 15:33

also having this problem, subscribe. I'd like to import legacy data that references blocked users and node import is blocking me from doing this.

Log in or register to post comments

Comment #8

DarrellDuane CreditAttribution: DarrellDuane commented 6 August 2009 at 15:54

As a workaround for this issue, I exported the CCK definition, searched for the UID field and edited the section referenceable_status so that it appears as below (I changed 0 => 0, to be 0 => 1).
I then deleted the CCK type and imported the edited definition. This User Reference field now accepts both blocked and unblocked users.

'referenceable_status' =>
array (
1 => 1,
0 => 1,
),