Skip to content

Commit 0b04364

Browse files
author
isayan
committed
bugfix Sendto sort order
1 parent 9b41a3f commit 0b04364

File tree

2 files changed

+11
-5
lines changed

2 files changed

+11
-5
lines changed

release/YaguraExtender-v2.2.jar

48 Bytes
Binary file not shown.

src/main/java/yagura/model/SendToMenu.java

Lines changed: 11 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,14 @@ private String getMenuItemCaption(boolean forceSortOrder, int ord, String captio
6666
else
6767
return caption;
6868
}
69-
69+
70+
private int getMenuItemCount(boolean isSubmenu) {
71+
if (isSubmenu)
72+
return this.mnuSendTo.getItemCount();
73+
else
74+
return this.menuList.size();
75+
}
76+
7077
public void renewMenu(SendToProperty property) {
7178
this.mnuSendTo.setText("Send To");
7279
this.sendToList.clear();
@@ -75,15 +82,14 @@ public void renewMenu(SendToProperty property) {
7582
this.mnuSendTo.removeAll();
7683
this.menuList.add(this.mnuSendTo);
7784
}
78-
BurpExtender.outPrintln("renewMenu:");
7985
List<SendToItem> sendToItemList = property.getSendToItemList();
8086
for (SendToItem item : sendToItemList) {
8187
if (item.isSelected()) {
8288
if (item.getExtend() != null) {
8389
SendToExtend sendToItem = new SendToExtend(item, this.invocation);
8490
if (sendToItem.getExtend() == SendToItem.ExtendType.PASTE_FROM_CLIPBOARD) {
8591
javax.swing.JMenu mnuItem = new javax.swing.JMenu();
86-
mnuItem.setText(getMenuItemCaption(property.isForceSortOrder(), this.menuList.size(), item.getCaption()));
92+
mnuItem.setText(getMenuItemCaption(property.isForceSortOrder(), getMenuItemCount(property.isSubMenu()), item.getCaption()));
8793
List<String> encodingList = BurpExtender.getInstance().getSelectEncodingList();
8894
for (String encoding : encodingList) {
8995
javax.swing.JMenuItem mnuItemEncoding = new javax.swing.JMenuItem();
@@ -102,7 +108,7 @@ public void renewMenu(SendToProperty property) {
102108
}
103109
} else {
104110
javax.swing.JMenuItem mnuItem = new javax.swing.JMenuItem();
105-
mnuItem.setText(getMenuItemCaption(property.isForceSortOrder(), this.menuList.size(), item.getCaption()));
111+
mnuItem.setText(getMenuItemCaption(property.isForceSortOrder(), getMenuItemCount(property.isSubMenu()), item.getCaption()));
106112
sendToList.add(sendToItem);
107113
mnuItem.addActionListener(sendToItem);
108114
if (property.isSubMenu()) {
@@ -117,7 +123,7 @@ public void renewMenu(SendToProperty property) {
117123
}
118124
} else {
119125
javax.swing.JMenuItem mnuItem = new javax.swing.JMenuItem();
120-
mnuItem.setText(getMenuItemCaption(property.isForceSortOrder(), this.menuList.size(), item.getCaption()));
126+
mnuItem.setText(getMenuItemCaption(property.isForceSortOrder(), getMenuItemCount(property.isSubMenu()), item.getCaption()));
121127
if (item.isServer()) {
122128
SendToMenuItem sendToItem = new SendToServer(item, this.invocation);
123129
sendToItem.addSendToListener(new SendToListener() {

0 commit comments

Comments
 (0)