`SamplePSReadLineProfile.ps1` SmartInsertQuote Error On Strict Mode

Aberta Para iniciantes
#5,121 0 comentários 0 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
2/5
Tempo estimado
1-3 horas
Facilidade para iniciantes
72/100
Tipo de issue
Bug
Clareza
Claramente especificada
Status de atividade
Pouca atividade
Stack de tecnologia
powershell
Domínio
cli

Direção de pesquisa

Comece em PSReadLine/SamplePSReadLineProfile.ps1, na condição SmartInsertQuote vinculada, e execute o perfil de exemplo sob Set-StrictMode -Version Latest. Reproduza o erro movendo-se para o final de uma linha e inserindo uma aspa; em seguida, verifique se a proteção trata essa posição, insere duas aspas e evita a exceção index-out-of-range.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

Needs-Triage :mag:
Prerequisites
  • Write a descriptive title.
  • Make sure you are able to repro it on the latest released version
  • Search the existing issues, especially the pinned issues.
Exception report
Exception             : 
    Type       : System.IndexOutOfRangeException
    TargetSite : 
        Name          : ThrowIndexOutOfRangeException
        DeclaringType : [Internal.Runtime.CompilerHelpers.ThrowHelpers]
        MemberType    : Method
        Module        : System.Private.CoreLib.dll
    Message    : Index was outside the bounds of the array.
    Source     : System.Private.CoreLib
    HResult    : -2146233080
    StackTrace : 
   at CallSite.Target(Closure, CallSite, Object, Object)
   at System.Management.Automation.EnumerableOps.SlicingIndex(Object target, IEnumerator indexes, Func`3 indexer)
   at <ScriptBlock>(Closure, FunctionContext)
CategoryInfo          : OperationStopped: (:) [], IndexOutOfRangeException
FullyQualifiedErrorId : System.IndexOutOfRangeException
InvocationInfo        : 
    ScriptLineNumber : 393
    OffsetInLine     : 17
    HistoryId        : 111
    ScriptName       : C:\Users\matt\OneDrive\Documents\PowerShell\profile.ps1
    Line             : if ($line[0..$cursor].Where{$_ -eq $quote}.Count % 2 -eq 1) {
                       
    Statement        : $line[0..$cursor].Where{$_ -eq $quote}.Count % 2 -eq 1
    PositionMessage  : At C:\Users\matt\OneDrive\Documents\PowerShell\profile.ps1:393 char:17
                       + …         if ($line[0..$cursor].Where{$_ -eq $quote}.Count % 2 -eq 1) { …
                       +               ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    PSScriptRoot     : C:\Users\matt\OneDrive\Documents\PowerShell
    PSCommandPath    : C:\Users\matt\OneDrive\Documents\PowerShell\profile.ps1
    CommandOrigin    : Internal
ScriptStackTrace      : at <ScriptBlock>, C:\Users\matt\.local\share\chezmoi\outchezmoiscripts\Sync-ScoopConfig.ps1: line 64
                        at <ScriptBlock>, <No file>: line 1
Screenshot

N/A

Environment data
PS Version: 7.6.0
PS HostName: Visual Studio Code Host
PSReadLine Version: 2.4.4-beta4
PSReadLine EditMode: Vi
OS: 10.0.22621.1 (WinBuild.160101.0800)
BufferWidth: 190
BufferHeight: 19
Steps to reproduce

Run the sample profile with Set-StrictMode -Version Latest . Then go to end of line. Then enter a quote.

Expected behavior

Two quotes should be entered

Actual behavior

Error happens.

Workaround

I edited the below line
https://github.com/PowerShell/PSReadLine/blob/caa53f8469e35f6ba6e1a434d4fc8efae2dde20c/PSReadLine/SamplePSReadLineProfile.ps1#L195
to add $cursor -lt $line.Length -and to beginning of the if block to short circuit and prevent the index out of range. I wasn't sure if $cursor - 1 inside the indexer would be appropriate as I don't completely understand all the code.

Linguagem predominante
C#
Estrelas
4.4k
Forks
341
Métricas de merge de PRs
Nenhum PR com merge em 30d

Guia de contribuição

Abrir o guia de contribuição

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de PowerShell/PSReadLine

Todas as issues de PowerShell/PSReadLine

Issues semelhantes

Mais issues de C#

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.