[NAS-2485] Definitions-add-event-seeds.jsp should ignore lines prefixed with # or // Created: 20/Jan/16 Updated: 01/Feb/16 Resolved: 27/Jan/16 |
|
Status: | Resolved |
Project: | NetarchiveSuite |
Component/s: | None |
Affects Version/s: | 5.1 |
Fix Version/s: | None |
Type: | Improvement | Priority: | Minor |
Reporter: | Søren Vejrup Carlsen (Inactive) | Assignee: | Søren Vejrup Carlsen (Inactive) |
Resolution: | Fixed | ||
Labels: | None | ||
Remaining Estimate: | Not Specified | ||
Time Spent: | 5m | ||
Original Estimate: | Not Specified |
Verification: | TEST1 1 verifies if you keep the comments in the seedlist |
Description |
Definitions-add-event-seeds.jsp should ignore lines prefixed with # or // These lines should just be skipped during ingest |
Comments |
Comment by Søren Vejrup Carlsen (Inactive) [ 01/Feb/16 ] |
Verified as part of TEST1 |
Comment by Søren Vejrup Carlsen (Inactive) [ 22/Jan/16 ] |
Now fixed. The fix also allows for Urls beginning with Uppercase HTTP:// or HTTPS:// . if (seed.length() != 0) { if (!(seed.startsWith("http://") || seed.startsWith("https://"))) { seed = "http://" + seed; to if (seed.length() != 0 && !seed.startsWith("#") && !seed.startsWith("//")) { // ignore empty lines and comments if (!(seed.toLowerCase().startsWith("http://") || seed.toLowerCase().startsWith("https://"))) { seed = "http://" + seed; |
Comment by Søren Vejrup Carlsen (Inactive) [ 20/Jan/16 ] |
More precisely, the method PartialHarvest.processSeed() needs to filter out all lines starting with // or # |
Comment by Søren Vejrup Carlsen (Inactive) [ 20/Jan/16 ] |
The methods to correct is the EventHarvestUtil.addconfiguration* |