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.

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

WW-4877: adds missing header with license #175

Merged
merged 21 commits into from
Oct 22, 2017
Merged

WW-4877: adds missing header with license #175

merged 21 commits into from
Oct 22, 2017

Conversation

lukaszlenart
Copy link
Member

@lukaszlenart lukaszlenart commented Oct 21, 2017

Each file must have a header with license see. Also this PR will enable checking missing headers on each build.

Resolves WW-4877 #

Copy link
Member

@sdutry sdutry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You weren't kidding when you said a lot of files were modified.

The only problem i found is the i18n resource files that changed encoding.

employee.id.required=ID mu� angegeben werden
employee.lastName.required=Nachname wird ben�tigt
employee.birthDate.required=Geburtsdatum wird ben�tigt
employee.backtolist=Zur�ck zur Mitarbeiterliste
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did the encoding of this file change?
Does it still show the translations correctly when running the application?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch, fixed :)

skill=Kenntnis
skill.name=Name
skill.description=Beschreibung

skill.name.required=Name muss angegeben werden
skill.backtolist=Zur�ck zur Kenntnis Liste
skill.backtolist=Zur�ck zur Kenntnis Liste
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same as before, encoding change?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed as well

@@ -29,9 +26,6 @@
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;

/**
* @see Anchor
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this javadoc reference no longer relevant?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Put back

@@ -1,4 +1,4 @@
<?xml version="1.0" encoding="ISO-8859-1"?>
<?xml version="1.0" encoding="UTF-8"?>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

import java.nio.file.Path;
import java.nio.file.Paths;

import javax.servlet.ServletInputStream;
import javax.servlet.http.HttpServlet;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

jsp.error.tagfile.nameNotUnique=
jsp.error.tagfile.nameFrom.noAttribute=
jsp.error.tagfile.nameFrom.badAttribute=
jsp.error.useBean.noSession=Il est ill�gal pour useBean d''utiliser une port�e de session (session scope) quand la page JSP indique (via la directive de page) qu''elle ne participe pas aux sessions
jsp.error.useBean.noSession=Il est ill�gal pour useBean d''utiliser une port�e de session (session scope) quand la page JSP indique (via la directive de page) qu''elle ne participe pas aux sessions
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

did the encoding of this file change? does it still show correctly?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

import org.osgi.framework.Bundle;
import org.osgi.framework.BundleContext;

import javax.servlet.ServletContext;
import java.util.List;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@@ -20,9 +18,6 @@
*/
package org.apache.struts2.oval.interceptor;

import net.sf.oval.constraint.NotNull;
import net.sf.oval.constraint.NotEmpty;
import net.sf.oval.constraint.Length;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

import com.opensymphony.module.sitemesh.DecoratorMapper;
import com.opensymphony.sitemesh.DecoratorSelector;
import com.opensymphony.sitemesh.Content;
import com.opensymphony.sitemesh.SiteMeshContext;
import com.opensymphony.sitemesh.compatability.Content2HTMLPage;
import com.opensymphony.sitemesh.compatability.OldDecorator2NewDecorator;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

import com.opensymphony.module.sitemesh.DecoratorMapper;
import com.opensymphony.sitemesh.DecoratorSelector;
import com.opensymphony.sitemesh.Content;
import com.opensymphony.sitemesh.SiteMeshContext;
import com.opensymphony.sitemesh.compatability.Content2HTMLPage;
import com.opensymphony.sitemesh.compatability.OldDecorator2NewDecorator;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@sdutry
Copy link
Member

sdutry commented Oct 22, 2017

LGTM 💯

@lukaszlenart
Copy link
Member Author

Great .... let it be ;-)

@lukaszlenart lukaszlenart merged commit 6e96f11 into apache:master Oct 22, 2017
@lukaszlenart lukaszlenart deleted the act-of-law branch October 22, 2017 07:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants