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

Unable to Push-publish new pages #18126

Closed
fabrizzio-dotCMS opened this issue Mar 10, 2020 · 2 comments
Closed

Unable to Push-publish new pages #18126

fabrizzio-dotCMS opened this issue Mar 10, 2020 · 2 comments

Comments

@fabrizzio-dotCMS
Copy link
Contributor

Describe the bug

Basically when Push-publishing New Pages
You'll see a NullPointer Exception that is originated in the class ContentHandler.java

by the following code:

Approx line 534:

// pushing live content requires some cleanup we have on contentletAPI.publish
	                if(implicitPublish) {
	                    content = contentletAPI.findContentletByIdentifier(content.getIdentifier(), false, content.getLanguageId(), systemUser, false);
	                    contentletAPI.publish(content, systemUser, false);
	                }

New Pages aren't found on the destination environment causing a NPE when publish is attempted.

Steps to reproduce the behavior:

  1. Set up a Push Publishing environment.
  2. Create an HTML Page, and push it to the receiver.
  3. The New Pages are not passing. instead, you'll see an NPE on the logs.

The error looks like this:

11:04:07.297  ERROR handler.ContentHandler - Error on File /Users/fabrizzio/code/servers/server2/assets/bundles/01E32PEDZAQ26QSBP5G68QJZRT/working/demo.dotcms.com, content id:(empty) :null
java.lang.NullPointerException: null
	at com.dotcms.content.elasticsearch.business.ESContentletAPIImpl.publish_aroundBody30(ESContentletAPIImpl.java:559) ~[dotcms_5.0.0_a933c1b.jar:?]
	at com.dotcms.content.elasticsearch.business.ESContentletAPIImpl$AjcClosure31.run(ESContentletAPIImpl.java:1) ~[dotcms_5.0.0_a933c1b.jar:?]

But I've seen it fail silently also.

Expected behavior

The system should be capable of Push-publishing Both New Pages as well as existing pages to the receiver.

Additional context

Add any other context about the problem here.

Acceptance Criteria

  1. Multilanguage
  2. Unit test
@erickgonzalez
Copy link
Contributor

@bryanboza
Copy link
Member

Fixed, tested on release-5.2.8 // Postgres // FF

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants