Update university-of-helsinki-faculty-of-theology.csl

[zuphilip]

(ref to #1301)
I don't know how I should change the other cases in the names

[zuphilip]

 <if variable="author">
<names variable="editor">
<label form="verb-short" text-case="capitalize-first" suffix=" "/>
<name prefix=" " and="text" initialize-with=".."/>
</names>
</if>

 <if variable="author editor" match="any">
<names variable="translator" suffix=".">
<label form="verb-short" text-case="capitalize-first" suffix=" "/>
<name prefix=" " and="text" initialize="false" initialize-with=""/>
</names>
</if>

                <if variable="container-author">
                  <group>
                    <names variable="container-author">
                      <label form="verb-short" prefix=" " suffix=" " strip-periods="true"/>
                      <name and="text" prefix=" " delimiter=", " initialize-with=". "/>
                    </names>
                  </group>
                </if>

[rmzelle]

You don't know what to do because you can't use a delimiter here?

I would keep the suffix on <label/> and get rid of the prefix on <name/>. A name label will only print if there is a name, but a label term might be empty/non-defined.

[adam3smith]

That's a really odd initialization think, but unfortunately we don't have a style guide...

[adam3smith]

this needs a semicolon in the suffix, since in the original there was one as part of the last group.

[zuphilip]

I think I mostly would appreciate if we can speak the changes through together. Okay? Let us maybe start with no. 3

                <if variable="container-author">
                  <group>
                    <names variable="container-author">
                      <label form="verb-short" prefix=" " suffix=" " strip-periods="true"/>
                      <name and="text" prefix=" " delimiter=", " initialize-with=". "/>
                    </names>
                  </group>
                </if>

• delete the prefix and let the suffix as it is
• the group is not needed, thus I can delete it
• the if is also not needed, right?
• anything else?

[adam3smith]

that's all correct.

[rmzelle]

For no. 3, it would be better to move the prefix on <label/> to <names/> (assuming it's needed).

[zuphilip]

Okay, I updated the other case in a similar spirit. Please have a look.

BTW I think the only case left for adjacent spaces is https://github.com/citation-style-language/styles/blob/master/spip-cite.csl#L67 . I am not sure what exactly this style is for. Is it okay as it is? Or should I delete the suffix=" " on this line?

[adam3smith]

it's for an online publishing system. Leave this as is.

[zuphilip]

Okay. Travis is happy here. Let me know what you think about the last one. Yes, I think this change here would close #1301 for now. 😆