Empty password [message #61598] |
Tue, 11 November 2008 10:52 |
Eclipse User |
|
|
|
Originally posted by: morten.hindsholm.dk
I am developing a connector, and I think I might have spotted a minor bug.
My backend requires a login, but it is legal to have an empty password.
However, AbstractRepositorySettingsPage.isPageComplete() explicitly checks -
via the private method isMissingCredentials() - against empty passwords.
This means that I cannot easily implement isPageComplete() in my
RepositorySettings class which extends AbstractRepositorySettingsPage, because
I cannot call super.isPageComplete() with an empty password, and I also cannot
bypass AbstractRepositorySettingsPage and call isPageComplete() in its
superclass AbstractTaskRepositoryPage. If you understand...
I think a reasonable fix would be to make
AbstractRepositorySettingsPage.isMissingCredentials() protected instead of
private.
--
/\/\orten
|
|
|
|
Re: Empty password [message #61681 is a reply to message #61657] |
Wed, 12 November 2008 10:04 |
Eclipse User |
|
|
|
Originally posted by: morten.hindsholm.dk
Steffen Pingel <steffen.pingel@tasktop.com> writes:
> Please file a new bug to request this API enhancement:
>
> http://eclipse.org/mylyn/
Bug #255003 filed.
> Thanks.
You're welcome :-)
>
> Steffen
>
>
> Morten Hindsholm wrote:
>
>> I am developing a connector, and I think I might have spotted a minor bug.
>>
>> My backend requires a login, but it is legal to have an empty password.
>>
>> However, AbstractRepositorySettingsPage.isPageComplete() explicitly checks
>> - via the private method isMissingCredentials() - against empty passwords.
>>
>> This means that I cannot easily implement isPageComplete() in my
>> RepositorySettings class which extends AbstractRepositorySettingsPage,
>> because I cannot call super.isPageComplete() with an empty password, and I
>> also cannot bypass AbstractRepositorySettingsPage and call
>> isPageComplete() in its superclass AbstractTaskRepositoryPage. If you
>> understand...
>>
>> I think a reasonable fix would be to make
>> AbstractRepositorySettingsPage.isMissingCredentials() protected instead of
>> private.
>>
>
--
/\/\orten
|
|
|
|
|
Powered by
FUDForum. Page generated in 0.03851 seconds