Skip to content
This repository has been archived by the owner on Apr 12, 2024. It is now read-only.

Check if "input" variable is defined #6819

Closed
wants to merge 1 commit into from
Closed

Conversation

renatus
Copy link
Contributor

@renatus renatus commented Mar 24, 2014

By default, "greeting" textfield in this example is prepopulated with "hello" text, but it's pretty easy to copy just filter code to use it in your app. If your textfield is empty while app loads, you'll get an error: "Error: [$interpolate:interr] Can't interpolate: Reverse: {{greeting|reverse}} TypeError: Cannot read property 'length' of undefined". To prevent this, we should check "input" variable, and proceed only in case it is defined.

By default, "greeting" textfield in this example is prepopulated with "hello" text, but it's pretty easy to copy just filter code to use it in your app. If your textfield is empty while app loads, you'll get an error: "Error: [$interpolate:interr] Can't interpolate: Reverse: {{greeting|reverse}} TypeError: Cannot read property 'length' of undefined". To prevent this, we should check "input" variable, and proceed only in case it is defined.
@btford btford added the gh: PR label Mar 24, 2014
@tbosch tbosch self-assigned this Mar 24, 2014
@tbosch tbosch closed this in 4a6d4de Mar 24, 2014
@tbosch tbosch removed their assignment Mar 24, 2014
tbosch pushed a commit that referenced this pull request Mar 24, 2014
By default, "greeting" textfield in this example is prepopulated with "hello" text, but it's pretty easy to copy just filter code to use it in your app. If your textfield is empty while app loads, you'll get an error: "Error: [$interpolate:interr] Can't interpolate: Reverse: {{greeting|reverse}} TypeError: Cannot read property 'length' of undefined". To prevent this, we should check "input" variable, and proceed only in case it is defined.

Closes #6819.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants