Hi

On Fri, Aug 30, 2013 at 4:18 PM, Dinesh Kumar <dinesh.ku...@enterprisedb.com
> wrote:

>
>>
>> - Changing the time, but not the date, on an existing expiration
>> datetime, doesn't generate SQL and enable the OK button. Changing just the
>> date does.
>>
>
> Apologies Dave. I am not able to explain you the problem properly. But
> below are my findings.
>
> Further to my observation, it's not generating the event of
>
> EVT_SPIN(XRCID("timValidUntil"), dlgRole::OnChangeSpin)
>
> which is in pg_Roles.cpp. If the spin event occurs on spin button, then
> it's directly going to
> "EVT_SPIN_x" in timespin.cpp. And also, i have observed that
> wxTimeSpinCtrl is our custom data type which we have been derived from the
> wxControl class. That may be the reason the spin control event is directly
> refering  to  timespin.cpp's EVT_SPIN_x functions. I have fixed this issue
> by appending an dlgRole's event to timespin.cpp's event and it is working
> fine.
>
> Kindly let me know if anything is unclear.
>

OK, that seems reasonable. Did you check if it breaks any other usage of
that control?

I wonder if, for 1.20, we should think about adding a 3rd party (or
creating a new) datetime control that we can use universally.


>
>
>>
>> - If I clear the date and time, SQL is not generated to reset the valid
>> until time to infinity.
>>
>>
> If the role's "rolvaliduntil" property is NULL or infinity then there is
> no password expiration for that user/role. I believe, in your case the
> "rolvaliduntil" might be the NULL. Hence, it's not generating any "VALID
> UNTIL 'infinity'" since, NULL ~ infinity and also we haven't changed
> anything. In the rest of the cases, i believe it will generate as you
> suggested.
>
> Kindly let me know if i miss anything here.
>

Yeah, I think you misunderstood me:

1) Create a new role, and set VALID UNTIL to some value. Close the dialogue.

2) Open the properties dialogue, then clear the date/time fields. That
should cause the dialogue to attempt to set VALID UNTIL to infinity, but
doesn't.

Thanks.

-- 
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

Reply via email to