Quantcast
Channel: YouTrackReSharper (RSRP) - Bug and Issue Tracker
Viewing all 106942 articles
Browse latest View live
↧

RSRP-471796: File member search by visibility incorrectly highlights unrelated parts of results

October 17, 2018, 5:18 am
$
0
0
Reporter Maria Pleskunina (Maria.Pleskunina) Maria Pleskunina (Maria.Pleskunina)
Created Oct 10, 2018 12:40:36 PM
Updated Oct 17, 2018 3:18:13 PM
Subsystem Navigation - Contextual
Assignee Alexander Ulitin (alexander.ulitin)
Priority Normal
State Submitted
Type Cosmetics
Fix version 2018.3
Affected versions 2018.2.3
Fixed In Version ReSharper Undefined
VsVersion All Versions

Some letters are still highlighted - e.g. when I search for “public”, I see letters like “p”, “l”, “i” highlighted.

↧
Search

RSRP-330237: Edit resource value on DisplayAttribute/ValidationAttribute

October 17, 2018, 5:26 am
$
0
0
Reporter Julien Lebosquain (mrjul) Julien Lebosquain (mrjul)
Created Sep 27, 2012 3:06:39 PM
Updated Oct 17, 2018 3:26:25 PM
Subsystem Localization (i18n, ResX)
Assignee Anton Spilnyy (Anton.Spilnyy)
Priority Normal
State Submitted
Type Feature
Fix version Backlog
Affected versions 7.0.1, 2018.3
Fixed In Version ReSharper Undefined
VsVersion All Versions
It's great that the context action Edit resource value has been added on static resource references.
However, it's missing on DisplayAttribute and ValidationAttribute that now have references in R# 7.

[Display(ResourceType = typeof(Resources), Name = "RoleAccessRead")]
Put the caret on RoleAccessRead, there is no context action for editing the resource value, forcing the use of the top level menu.
↧
↧

RSRP-331245: Surround With context menu should be aware of valid combinations

October 17, 2018, 5:30 am
$
0
0
Reporter Mackenzie Zastrow (Mackenzie.Zastrow) Mackenzie Zastrow (Mackenzie.Zastrow)
Created Oct 11, 2012 5:54:29 PM
Updated Oct 17, 2018 3:30:15 PM
Subsystem Context Actions
Assignee Alisa Afonina (alisa.afonina)
Priority Normal
State Submitted
Type Bug
Fix version Backlog
Affected versions 7.1 EAP, 7.0.1, 2018.3
Fixed In Version ReSharper Undefined
VsVersion All Versions
When selecting class elements (methods, properties, fields, etc), the icon for "Surround With", when clicked, shows a default of "Surround with {}" which is invalid. At a minimum, it should hide the "Surround with {}" and ideally default to "Surround with #region"
↧

RSRP-333099: Add context actions for detected parameters, types, type arguments, language keywords etc. in comments

October 17, 2018, 5:32 am
$
0
0
Reporter Daniel Hilgarth (DanielH) Daniel Hilgarth (DanielH)
Created Oct 25, 2012 2:42:56 PM
Updated Oct 17, 2018 3:32:02 PM
Subsystem Context Actions
Assignee Alisa Afonina (alisa.afonina)
Priority Normal
State Submitted
Type Feature
Fix version Backlog
Affected versions 2018.3
Fixed In Version ReSharper Undefined
VsVersion All Versions
Assume a comment like this:

/// <param name="info">...</param>
/// <exception cref="ArgumentNullException">The info parameter is null.</exception>

I would like to have a context action for info that offers me to change it into <paramref name="info"/>.
I would like to have a context action for null that offers me to change it into <c>null</c>.
↧

RSRP-471129: Create quickfixes for altering VB language level as we have in C#

October 17, 2018, 5:32 am
$
0
0
Reporter Sergey Kuks (coox) Sergey Kuks (coox)
Created Aug 17, 2018 4:44:37 PM
Updated Oct 17, 2018 3:32:14 PM
Subsystem Platform - VS Integration
Assignee Andrew Karpov (andrew.karpov)
Priority Normal
State In Progress
Type Feature
Fix version Backlog
Affected versions No Affected versions
Fixed In Version ReSharper Undefined
VsVersion All Versions
↧
↧

RSRP-334651: Rename variable is not the first option when you do rename and hit Alt + Enter

October 17, 2018, 5:33 am
$
0
0
Reporter Ilya Chernomordik (Ilya.Chernomordik) Ilya Chernomordik (Ilya.Chernomordik)
Created Nov 6, 2012 7:33:11 PM
Updated Oct 17, 2018 3:33:24 PM
Resolved Oct 17, 2018 3:33:24 PM
Subsystem Context Actions
Assignee Alisa Afonina (alisa.afonina)
Priority Normal
State Fixed
Type Usability Problem
Fix version Unidentified prior version
Affected versions No Affected versions
Fixed In Version ReSharper Undefined
VsVersion All Versions
I have posted a bug recently that Apply renaming refactoring does not work always as it should (RSRP-332956).
But when it works then it's also not ideal from UX point of view. For instance you have a local variable long abc = 34;
Then you type d inside to be abdc and then hit Alt+Enter. It does not always allow you to do the renaming refactoring (RSRP-332956), but when it does it's not the first option in the menu so you cannot just hit enter. The first option is to delete unused variable which is quite strange. It's quite obvious that if you have a renaming refactoring then you don't want to delete it (or at least it's very unlikely). Could you please make it the first item + selected by default.
↧

RSRP-337107: ReSharper contextual insertion of "using" does not follow the code style

October 17, 2018, 5:37 am
$
0
0
Reporter Martin Cetkovský (mcetkovsky) Martin Cetkovský (mcetkovsky)
Created Jan 9, 2013 12:43:30 PM
Updated Oct 17, 2018 3:37:30 PM
Resolved Oct 17, 2018 3:37:30 PM
Subsystem Context Actions
Assignee Alisa Afonina (alisa.afonina)
Priority Normal
State Fixed
Type Bug
Fix version Unidentified prior version
Affected versions 7.1.1
Fixed In Version ReSharper Undefined
VsVersion All Versions
Consider the following code:
namespace SomeNamespace
{
using System;

public class SomeClass
{
}
}

Please note that the "using" is inside the namespace (just for curiosity, it corresponds to the SA1200 StyleCop rule)

Change the "public class" line to the following:
public class SomeClass : Stack<int>

ReSharper should offer you inserting a respective namespace as a using clause (CTRL+ENTER or clicking in the pop-up).

However, ReSharper inserts the "using" clause outside the namespace instead inside it as it would be expected.

What will happen:
using System.Collections.Generic;

namespace SomeNamespace
{
using System;

public class SomeClass : Stack<int>
{
}
}

What is expected:
namespace SomeNamespace
{
using System;
using System.Collections.Generic;

public class SomeClass : Stack<int>
{
}
}
↧

RSRP-337939: An inspection that detects possible ArgumentOutOfRangeException

October 17, 2018, 5:39 am
$
0
0
Reporter Dmitry Matveev (Dmitry.Matveev) Dmitry Matveev (Dmitry.Matveev)
Created Jan 31, 2013 4:11:27 PM
Updated Oct 17, 2018 3:39:04 PM
Subsystem Context Actions
Assignee Alisa Afonina (alisa.afonina)
Priority Normal
State Submitted
Type Feature
Fix version Backlog
Affected versions 2018.3
Fixed In Version ReSharper Undefined
VsVersion All Versions
One use case for such inspection is String.Remove() arguments:

const string str = "string";
// we wrote some code and forgot what is str
//
// If we fail to check String.Remove() arguments, we can get an ArgumentOutOfRangeException:
string remove = str.Remove(0, 10);
but there may be other cases
↧
Search

RSRP-339420: Add context action IDictionary.ContainsKey() + IDictionary.Item() to IDictionary.TryGetValue()

October 17, 2018, 5:41 am
$
0
0
Reporter Alexander Zaytsev (hazzik) Alexander Zaytsev (hazzik)
Created Mar 4, 2013 4:59:53 AM
Updated Oct 17, 2018 3:41:04 PM
Resolved Oct 17, 2018 3:41:04 PM
Subsystem Context Actions
Assignee Alisa Afonina (alisa.afonina)
Priority Normal
State Duplicate
Type Feature
Fix version Backlog
Affected versions 7.1
Fixed In Version ReSharper Undefined
VsVersion All Versions
Replace follwing code

if (dictionary.ContainsKey(key)) {
    var value = dictionary[key];
    // other
}

with this

$TValue$ value;
if (dictionary.TryGetValue(key, out result) {
    // other
}

I have about 16 different patterns but they still do not handle all the cases, so I think it would be useful to have this patterns with R# core.
↧
↧

RSRP-461907: Feature: ContainsKey followed by lookup to TryGetValue

October 17, 2018, 5:41 am
$
0
0
Reporter Edward Avis (epa) Edward Avis (epa)
Created Nov 24, 2016 6:38:26 PM
Updated Oct 17, 2018 3:41:04 PM
Subsystem Code Analysis - C#
Assignee Alexander Shvedov (shvedov)
Priority Normal
State Open
Type Bug
Fix version Backlog
Affected versions No Affected versions
Fixed In Version ReSharper Undefined
VsVersion All Versions 
            Dictionary<string, int> mydict = ...;
            if (mydict.ContainsKey(name))
                return mydict[name];

This could be better written as

            Dictionary<string, int> mydict = ...;
            string r;
            if (mydict.TryGetValue(name, out r))
                return r;

Resharper should offer to convert it automatically.
↧

RSRP-471786: Missing suggestion when correcting a typo

October 17, 2018, 5:49 am
$
0
0
Reporter Igor Akhmetov (Igor.Akhmetov) Igor Akhmetov (Igor.Akhmetov)
Created Oct 8, 2018 7:08:48 PM
Updated Oct 17, 2018 3:49:29 PM
Subsystem ReSpeller
Assignee Artem Bukhonov (Artem.Bukhonov)
Priority Normal
State Submitted
Type Bug
Fix version Backlog
Affected versions No Affected versions
Fixed In Version ReSharper Undefined
VsVersion All Versions
When correcting "mamber" R# suggests "amber" but not "member".
↧

RSRP-471785: Respeller suggestions

October 17, 2018, 5:51 am
$
0
0
Reporter Simon Hewitt (simmotech) Simon Hewitt (simmotech)
Created Oct 8, 2018 5:56:42 PM
Updated Oct 17, 2018 3:51:47 PM
Subsystem ReSpeller
Assignee Artem Bukhonov (Artem.Bukhonov)
Priority Normal
State Submitted
Type Improvement
Fix version Backlog
Affected versions No Affected versions
Fixed In Version ReSharper Undefined
VsVersion All Versions
1) More granularity in which identifiers are checked. For example, I would prefer not to check lambda variables as they are usually abbreviations. e.g. "risi" in this example
                itemStatusLookup = ResourceItemHelper.GetResourceItemStatusInfos()
                    .ToDictionary(risi => risi.ItemID);

2) Allow me to set the ordering so "This Computer" is the first choice - reduces number of keystrokes when adding to a dictionary.

3) A button on the Respeller settings form (after "User words") to sort the user words list case insensitive. Makes it easier to find whether a word has been included or not.

4) A bit of a long shot but an additional "Group by" type in Inspection Results would be useful. Similar to "Issue Type" but grouping by the text. So for an "Type in identifier Issues in 'XXX', it would show "Typo in identifier" at the top level and then ["Typo in identifier: "trimmable"] at the next level with the occurrence count shown in brackets like some other "Group by"s do. Then the final level is the actual File locations.

PS Is this code the very same Respeller Pro I bought as an add-on some 4/5 years ago?
↧

RSRP-471764: Creating class and retyping it to enum makes it reload

October 17, 2018, 5:52 am
$
0
0
Reporter Andrey Simukov (Andrey.Simukov) Andrey Simukov (Andrey.Simukov)
Created Oct 5, 2018 4:59:03 PM
Updated Oct 17, 2018 3:52:35 PM
Subsystem Platform - VS Integration
Assignee Serge Baltic (baltic)
Priority Show-stopper
State Open
Type Bug
Fix version 2018.3
Affected versions 2018.2.3
Fixed In Version ReSharper Undefined
VsVersion All Versions
1) Locate shared project
2) Create class through Add / New Item / Class - call it "TestEnum"
3) Observe "TestEnum.cs" being created
4) Rewrite the "class" part to "public enum" so you get "public enum TestEnum"
5) Start typing enum values
6) Observe that a dialog to reload the file is presented.
7) Click "No"
8) The dialog is then presented again

Disabling ReSharper solves this problem
↧
↧

RSRP-471780: 'Invert if statement to reduce nesting' inspection isn't shown in CLT results

October 17, 2018, 5:53 am
$
0
0
Reporter Angelina Elycheva (Angelina.Elycheva) Angelina Elycheva (Angelina.Elycheva)
Created Oct 8, 2018 1:19:00 AM
Updated Oct 17, 2018 3:53:05 PM
Subsystem ReSharper Automation Tools (Command Line)
Assignee Andrey Dyatlov (Andrey.Dyatlov)
Priority Normal
State Open
Type Bug
Fix version Backlog
Affected versions No Affected versions
Fixed In Version ReSharper Undefined
VsVersion All Versions
0. Try this code:
public void TestMethod()
        {
            if (MyObject.MyBool)
            {
                if (MyObject.MyString != string.Empty)
                {
                    Console.WriteLine(MyObject.MyString);
                }
            }
        }
1. Inspect code issues for solution in ReSharper and by CLT

Result: ReSharper shows '"Invert 'if' statement to reduce nesting" while CLT does not.
'
↧

RSRP-471773: Find Usages shows strange things in external sources

October 17, 2018, 5:54 am
$
0
0
Reporter Ivan Migalev (Ivan.Migalev) Ivan Migalev (Ivan.Migalev)
Created Oct 6, 2018 4:42:43 PM
Updated Oct 17, 2018 3:54:50 PM
Subsystem Navigation - Search (Find Usages)
Assignee Alexander Ulitin (alexander.ulitin)
Priority Normal
State Submitted
Type Improvement
Fix version Backlog
Affected versions 2018.2.3
Fixed In Version ReSharper Undefined
VsVersion All Versions
(Initially found in Rider, but works the same in ReSharper.)

1. Open any decompiled class, e.g. System.Net.Http.SocketsHttpHandler from .NET Core.
2. Focus any instance field (e.g. _handler).
3. Press Find Usages key (Alt-F7 on my machine).
4. Rider actually finds usages, but shows the results in a strange way (probably because they're linked with the compiled code).


The code is already decompiled, so I expect that Rider will show actual source code lines instead of this.

It works in Visual Studio with ReSharper the same way, here's a sample from .NET Framework's String.m_firstChar:

↧
Search

RSRP-471770: Make the "Build already running" options more understandable

October 17, 2018, 5:55 am
$
0
0
Reporter Richard Ginzburg (rickythefox) Richard Ginzburg (rickythefox)
Created Oct 5, 2018 9:59:37 PM
Updated Oct 17, 2018 3:55:15 PM
Subsystem Solution builder
Assignee Anton Spilnyy (Anton.Spilnyy)
Priority Normal
State Submitted
Type Usability Problem
Fix version 2018.3
Affected versions No Affected versions
Fixed In Version ReSharper Undefined
VsVersion All Versions

This dialog is really ambiguous.
Abort = abort the operation you are trying to do (start a build) or abort the running build?
Ignore = ignore the running build? Ignore what?

Fits right into the Interface hall of shame: http://hallofshame.gp.co.at/errormsg.htm

↧

RSRP-471763: Remove braces: Removes first brace only!

October 17, 2018, 5:56 am
$
0
0
Reporter Marcel (aureole82) Marcel (aureole82)
Created Oct 5, 2018 2:51:41 PM
Updated Oct 17, 2018 3:56:05 PM
Subsystem Context Actions
Assignee Andrew Karpov (andrew.karpov)
Priority Critical
State Submitted
Type Bug
Fix version 2018.3
Affected versions No Affected versions
Fixed In Version ReSharper Undefined
VsVersion All Versions

Remove braces breaks code:
Remove braces breaks code

Microsoft Visual Studio Community 2017 Version 15.8.6
JetBrains ReSharper Ultimate 2018.2.3 Build 182.0.20180912.70621

↧
↧

RSRP-345933: Context action: intialize collection fields with empty collection/ new collection

October 17, 2018, 6:03 am
$
0
0
Reporter Sergey Kuks (coox) Sergey Kuks (coox)
Created Mar 19, 2013 5:47:29 PM
Updated Oct 17, 2018 4:03:41 PM
Resolved Oct 17, 2018 4:03:41 PM
Subsystem Context Actions
Assignee Alisa Afonina (alisa.afonina)
Priority Major
State Duplicate
Type Feature
Fix version Backlog
Affected versions No Affected versions
Fixed In Version ReSharper Undefined
VsVersion All Versions
we have message types (DTOs). it would be nice if Resharper could in the constructor initialize all collection properties to empty collections automatically with a single shortcut
↧

RSRP-337986: Add a default property initialization to constructor

October 17, 2018, 6:03 am
$
0
0
Reporter Bojan Vrhovnik (bojanv) Bojan Vrhovnik (bojanv)
Created Feb 1, 2013 4:33:45 PM
Updated Oct 17, 2018 4:03:41 PM
Subsystem Quick Fixes
Assignee Alisa Afonina (alisa.afonina)
Priority Major
State Submitted
Type Feature
Fix version Backlog
Affected versions 2018.3
Fixed In Version ReSharper Undefined
VsVersion All Versions
Hi all!

My suggestion would be to add a default property initialization to a constructor (or multiple)from quick action.

For example:
public string Query{get;set;}
public List<MyClass> MyList{get;set;}

Press ALT+ENTER on the query and it will create a constructor (if not present - default) or add a default initialization to the existing one.

result:

public MyClassName(){
Query = "";
MyList = new List<MyClass>();
}

This functionality would be awesome. I tryed to make my own plugin, but had problem with reference types. Works great with simple value types though.
↧

RSRP-471794: Improvements to "Remove Unused References": recognize references to assemblies in app.config

October 17, 2018, 6:05 am
$
0
0
Reporter Olga Rodygina (olgarodygina) Olga Rodygina (olgarodygina)
Created Oct 10, 2018 12:07:23 PM
Updated Oct 17, 2018 4:05:15 PM
Subsystem Optimize References
Assignee Alexander Ulitin (alexander.ulitin)
Priority Normal
State Submitted
Type Bug
Fix version Backlog
Affected versions 2018.2.3
Fixed In Version ReSharper Undefined
VsVersion All Versions
I have noticed that now resharper recognizes references to assembblies in config files, and that is great!
there is still room for improvment - it recognized the Vallstein.Encoder reference in the follwing (for the first time)

<extensions>
<bindingElementExtensions>
<add name="compactMessageEncoding"
type="Vallstein.Encoder.CompactMessageEncodingElement, Vallstein.Encoder, Version=1.0.0.0, Culture=neutral, PublicKeyToken=ee70f91902509a02"/>
</bindingElementExtensions>
</extensions>

in a web.config file, but not the entirely similar

<add initializeData="c:\logs\front9bankmsg.svclog"
type="Vallstein.Trace.LimitedTraceListener, Vallstein.Trace, Version=1.0.0.0, Culture=neutral, PublicKeyToken=ee70f91902509a02"
name="ServiceModelMessageLoggingListener"
traceOutputOptions="None">
<filter type="Vallstein.Trace.MsgFilter, Vallstein.Trace, Version=1.0.0.0, Culture=neutral, PublicKeyToken=ee70f91902509a02" />
</add>

in an app.config file, and also for similar references in other app.config files
↧
Viewing all 106942 articles
Browse latest View live
Search
<script src="https://jsc.adskeeper.com/r/s/rssing.com.1596347.js" async> </script>