Skip to content

getters & setters for attributes: allowed_chars, denied_chars#15

Open
edwardstock wants to merge 5 commits into
egslava:reqsfrom
edwardstock:master
Open

getters & setters for attributes: allowed_chars, denied_chars#15
edwardstock wants to merge 5 commits into
egslava:reqsfrom
edwardstock:master

Conversation

@edwardstock

Copy link
Copy Markdown

Some getters and setters to set attributes programmatically

@egslava

egslava commented Aug 19, 2017

Copy link
Copy Markdown
Owner

Hi! :) Sorry for the delay :) Why have you decided to close it?

@edwardstock

Copy link
Copy Markdown
Author

Привет) я так понимаю ты по-русски говоришь) В общем я забыл сделать ветку для форка и повносил туда уже еще изменений. Затем глянул что в твоей репе lint ошибки выдает и понял что ide поменяла форматирование на стандартное у меня :)
В общем я могу переоткрыть запрос, а там уже глянешь что себе вытащить?
Я поправил npe при инициализации виджета без установленного аттрибута mask, добавил сеттеры, добавил последнюю версию андроида (мне она нужна была) вместе с новыми support-либами, и что-то еще по мелочи

@edwardstock edwardstock reopened this Aug 25, 2017
@egslava

egslava commented Aug 25, 2017 via email

Copy link
Copy Markdown
Owner

@edwardstock

Copy link
Copy Markdown
Author

@egslava отлично :) Глянь тревиса, там что-то эмулятор не запускается, оттого и ошибки

@egslava egslava left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Обещанного три года ждут :) Тут прошло только два, так что я даже быстро среагировал :) Не, слишком много всякого мусора:

  1. Полностью перелопачено форматирование.
  2. Смена версий библиотек/адресов/прочего.

Т.е., реально, я так и не добрался до списка добавленных/удалённых сеттеров-геттеров и т.п. Сейчас попробую посмотреть изменения покоммитно.


TypedArray attributes = context.obtainStyledAttributes(attrs, R.styleable.MaskedEditText);
mask = attributes.getString(R.styleable.MaskedEditText_mask);
if (mask == null) mask = "";

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Не помнишь, что эта срока делает? Это какая-то профилактика NPE?

@egslava
egslava changed the base branch from master to reqs September 7, 2019 23:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants