xamarin/Xamarin.Forms

Binding compiler fails with indexers on Arrays

Aberta

#4.524 aberto em 23 de nov. de 2018

Ver no GitHubGuia do colaborador
 (3 comentários) (1 reação) (0 responsável)C# (1.926 forks)batch import
a/Xaml </>a/binding ⛓e/5 :clock5:help wantedin-progressinactivet/bug :bug:up-for-grabs

Métricas do repositório

Stars
 (5.644 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

Description

I was creating the repro for #4516, and found out that this Xaml fail compilation.

<?xml version="1.0" encoding="UTF-8"?>
<ContentPage
		xmlns="http://xamarin.com/schemas/2014/forms"
		xmlns:x="http://schemas.microsoft.com/winfx/2009/xaml"
		xmlns:local="using:Xamarin.Forms.Xaml.UnitTests"
		x:Class="Xamarin.Forms.Xaml.UnitTests.Gh4516"
		x:DataType="local:Gh4516VM">
	<Image x:Name="image" Source="{Binding Images[0]}" />
</ContentPage>

Expected Behavior

should compile, and works

Actual Behavior

throw a NRE at compilation

Basic Information

  • Version with issue: master

Guia do colaborador

Direção de pesquisa
Reproduza a NullReferenceException usando o trecho XAML fornecido. Em seguida, rastreie o caminho do código do compilador de vinculação para encontrar onde a indexação de array é tratada. Depure para identificar a origem da referência nula.
Pilha de tecnologia
csharp
Domain
mobile
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C#XAMLXamarin.Forms
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40