Skip to content

Commit 0e35e22

Browse files
softommywumpz
authored andcommitted
fix JSQLParser/JSqlParser #679 (#703)
1 parent 4854438 commit 0e35e22

File tree

2 files changed

+15
-18
lines changed

2 files changed

+15
-18
lines changed

src/main/jjtree/net/sf/jsqlparser/parser/JSqlParserCC.jjt

Lines changed: 14 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -3593,7 +3593,7 @@ AlterExpression AlterExpression():
35933593
List<String> columnNames = null;
35943594
List<ConstraintState> constraints = null;
35953595
ForeignKeyIndex fkIndex = null;
3596-
NamedConstraint index = null;
3596+
Index index = null;
35973597
Table fkTable = null;
35983598
AlterExpression.ColumnDataType alterExpressionColumnDataType = null;
35993599
}
@@ -3604,20 +3604,19 @@ AlterExpression AlterExpression():
36043604
(
36053605
LOOKAHEAD(2) ( <K_PRIMARY> <K_KEY> columnNames=ColumnsNamesList() { alterExp.setPkColumns(columnNames); } ) constraints=AlterExpressionConstraintState() { alterExp.setConstraints(constraints); }
36063606
|
3607-
LOOKAHEAD(2) (
3608-
tk=<K_INDEX>
3609-
sk3=ColumnsNamesListItem()
3610-
{
3611-
columnNames = new ArrayList();
3612-
columnNames.add(sk3);
3613-
3614-
index = new NamedConstraint();
3615-
index.setType(tk.image);
3616-
index.setColumnsNames(columnNames);
3617-
alterExp.setIndex(index);
3618-
}
3619-
constraints=AlterExpressionConstraintState() { alterExp.setConstraints(constraints); }
3620-
)
3607+
LOOKAHEAD(2) (
3608+
tk = < K_INDEX >
3609+
sk3 = RelObjectName()
3610+
columnNames = ColumnsNamesList()
3611+
{
3612+
index = new Index();
3613+
index.setType(tk.image);
3614+
index.setName(sk3);
3615+
index.setColumnsNames(columnNames);
3616+
alterExp.setIndex(index);
3617+
}
3618+
constraints=AlterExpressionConstraintState() { alterExp.setConstraints(constraints); }
3619+
)
36213620
|
36223621
( (LOOKAHEAD(2) <K_COLUMN>)?
36233622
alterExpressionColumnDataType = AlterExpressionColumnDataType() { alterExp.addColDataType(alterExpressionColumnDataType); }

src/test/java/net/sf/jsqlparser/statement/alter/AlterTest.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@
1212
import static org.junit.Assert.assertNotNull;
1313
import static org.junit.Assert.assertNull;
1414
import static org.junit.Assert.assertTrue;
15-
import org.junit.Ignore;
1615
import org.junit.Test;
1716

1817
public class AlterTest {
@@ -313,8 +312,7 @@ public void testIssue633() throws JSQLParserException {
313312
}
314313

315314
@Test
316-
@Ignore
317315
public void testIssue679() throws JSQLParserException {
318-
assertSqlCanBeParsedAndDeparsed("ALTER TABLE tb_session_status ADD INDEX idx_user_id(user_id)");
316+
assertSqlCanBeParsedAndDeparsed("ALTER TABLE tb_session_status ADD INDEX idx_user_id_name (user_id, user_name(10)), ADD INDEX idx_user_name (user_name)");
319317
}
320318
}

0 commit comments

Comments
 (0)