Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[DS-1065] controlled-vocabulary.xsd says that the 'id' attribute is optional, so make it optional instead of NPE if missing. #35

Merged
merged 5 commits into from Jul 9, 2012
Merged

[DS-1065] controlled-vocabulary.xsd says that the 'id' attribute is optional, so make it optional instead of NPE if missing. #35

merged 5 commits into from Jul 9, 2012

Conversation

mwoodiupui
Copy link
Member

This consists of four commits:

  1. I had to reformat some hard tabs to be able to read the code.
  2. The actual fix.
  3. A simple mock ConfigurationManager so I can test.
  4. The actual unit test and some scaffolding.

mwoodiupui added a commit that referenced this pull request Jul 9, 2012
@mwoodiupui
Merge pull request #35 from mwoodiupui/DS-1065
e9fbe19 
[DS-1065] controlled-vocabulary.xsd says that the 'id' attribute is optional, so make it optional instead of NPE if missing.
@mwoodiupui mwoodiupui merged commit e9fbe19 into DSpace:master Jul 9, 2012
el95149 pushed a commit to el95149/HEAL-DSpace that referenced this pull request Apr 4, 2014
@gkarka
Solr schema fixes for greek accent insensitive search. Closes DSpace#35
6d45fdd
el95149 pushed a commit to el95149/HEAL-DSpace that referenced this pull request May 14, 2014
@gkarka
Solr schema fixes for greek accent insensitive search. Closes DSpace#35
45d323c
artlowel referenced this pull request in atmire/DSpace Jun 13, 2014
@mwoodiupui
Merge pull request #35 from mwoodiupui/DS-1065
d6d28a6 
[DS-1065] controlled-vocabulary.xsd says that the 'id' attribute is optional, so make it optional instead of NPE if missing.
hardyoyo pushed a commit to hardyoyo/DSpace that referenced this pull request Aug 26, 2015
Merge pull request DSpace#35 from hardyoyo/lso-1457-MST-edits
b823102 
LSO DSpace#1457 MST edits
olli-gold pushed a commit to tubhh/DSpace that referenced this pull request Oct 22, 2019
@abollini
Merged in tubdok-4science-dev-new-CST-1815 (pull request DSpace#35)
c20612e 
Tubdok 4science dev new CST-1815
dspeed2 added a commit to UoEMainLibrary/DSpace that referenced this pull request Nov 7, 2019
@dspeed2
Merge pull request DSpace#35 from edina/issue22274
90c692d 
Issue22274 - Fix for Broken link on DataShare homepage to UoE homepage
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@mwoodiupui