[ 
https://issues.apache.org/jira/browse/NETBEANS-5399?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17294554#comment-17294554
 ] 

Christian Lenz edited comment on NETBEANS-5399 at 3/3/21, 2:29 PM:
-------------------------------------------------------------------

Reproducible, thx :)


was (Author: chrizzly):
Reproducable, thx :)

> JavaScript: rename variable - not all occurrences found
> -------------------------------------------------------
>
>                 Key: NETBEANS-5399
>                 URL: https://issues.apache.org/jira/browse/NETBEANS-5399
>             Project: NetBeans
>          Issue Type: Bug
>          Components: javascript - Refactoring
>    Affects Versions: 12.3
>         Environment: Product Version: Apache NetBeans IDE 12.3-beta2
> Java: 14.0.2; Java HotSpot(TM) 64-Bit Server VM 14.0.2+12-46
> Runtime: Java(TM) SE Runtime Environment 14.0.2+12-46
>            Reporter: Czukowski
>            Priority: Minor
>
> Consider a sample project consisting of a single file, {{main.js}}:
> {code}
> 'use strict';
> const registry = {
>     categories: {},
> };
> const f = function (category, event, element = null) {
>     const obj = registry.categories[category]; // [1]
>     if (!obj.hasOwnProperty(event)) { // [2]
>         obj[event] = true; // [3]
>         element.addEventListener(event, function () {
>             // ...
>         });
>     }
> };
> {code}
> The variable in question is {{obj}}, referenced at three places. Try renaming 
> it by placing a cursor at its occurrences and pressing Ctrl+R. Now, depending 
> on where a cursor has been placed, there is a different outcome:
> - If the cursor is at [1] or [3], the occurrence at [2] will not be selected 
> and therefore renamed
> - If the cursor is at [2], all three occurrences will be selected
> Commenting out or removing some other code 'fixes' it, in a way that renaming 
> the variable at any of the three occurrences will rename all three:
> - Removing {{'use strict'}} at the 1st line
> - Removing the default {{null}} value for the {{element}} argument at the 
> function {{f}} declaration
> - Removing the whole call to {{element.addEventListener}}
> The desired behavior is that all occurrences are correctly selected without 
> the need to remove any code.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@netbeans.apache.org
For additional commands, e-mail: commits-h...@netbeans.apache.org

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists

Reply via email to