Skip to content
This repository was archived by the owner on Dec 19, 2018. It is now read-only.

Added temporary parse error for helper directive #531

Merged
merged 1 commit into from
Sep 21, 2015
Merged

Conversation

ajaybhargavb
Copy link
Contributor

@@ -425,4 +425,7 @@ Instead, wrap the contents of the block in "{{}}":
<data name="TagHelperParseTreeRewriter_InvalidNestedTag" xml:space="preserve">
<value>The &amp;lt;{0}&amp;gt; tag is not allowed by parent &amp;lt;{1}&amp;gt; tag helper. Only child tag helper(s) targeting tag name(s) '{2}' are allowed.</value>
</data>
<data name="ParseError_HelperDirectiveNotAvailable" xml:space="preserve">
<value>The @helper directive is not available in Razor 4.</value>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Suggestions for error message welcome 😄

Choose a reason for hiding this comment

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

The {0} directive is no longer supported.

Don't want to specify the version for when we rev, and also don't want to inline @helper because it may be localized.

@NTaylorMullen
Copy link

@ajaybhargavb
Copy link
Contributor Author

Updated.

@@ -425,4 +425,7 @@ Instead, wrap the contents of the block in "{{}}":
<data name="TagHelperParseTreeRewriter_InvalidNestedTag" xml:space="preserve">
<value>The &amp;lt;{0}&amp;gt; tag is not allowed by parent &amp;lt;{1}&amp;gt; tag helper. Only child tag helper(s) targeting tag name(s) '{2}' are allowed.</value>
</data>
<data name="ParseError_HelperDirectiveNotAvailable" xml:space="preserve">
<value>The {0} directive is no longer supported.</value>
Copy link
Contributor

Choose a reason for hiding this comment

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

is not supported

@NTaylorMullen
Copy link

:shipit: once @Eilon's comment is addressed.

@ajaybhargavb ajaybhargavb merged commit 153ed57 into dev Sep 21, 2015
@ajaybhargavb ajaybhargavb deleted the helper-error branch September 21, 2015 19:12
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.

4 participants