Context Navigation

← Previous Ticket
Next Ticket →

#36631 closed enhancement (fixed)

wp.template() should throw an error if the template is not found

Reported by:	joehoyle	Owned by:	desrosj
Priority:	normal	Milestone:	6.1
Component:	General	Version:
Severity:	normal	Keywords:	has-patch needs-testing
Cc:		Focuses:	javascript

Description

When calling wp.template() in JS, if the template element does not exist (maybe you forgot to wait for document.ready) a very cryptic error happens ("Uncaught TypeError: Cannot read property 'replace' of undefined"). I'm not totally familiar with how defensive we typically are for things like this - but in this case it would have saved me a good deal of effort if I saw a descriptive error.

Attachments (2)

36631.diff (506 bytes ) - added by joehoyle 10 years ago.
36631.2.diff (499 bytes ) - added by desrosj 4 years ago.

Download all attachments as: .zip

Change History (11)

@joehoyle
10 years ago

Attachment 36631.diff added

This ticket was mentioned in Slack in #core by talldanwp. View the logs.

6 years ago

#2 @noisysocks
6 years ago

Keywords needs-refresh added

Good suggestion @joehoyle! The patch doesn't apply cleanly anymore, but if you or someone else is happy to refresh it then I think let's get this in.

#3 @desrosj
4 years ago

Keywords 2nd-opinion needs-refresh removed
Milestone → 6.1

Found this one while going through tickets missing a milestone. Adding an updated patch.

@noisysocks Care to give me a sanity check?

@desrosj
4 years ago

Attachment 36631.2.diff added

This ticket was mentioned in Slack in #core by chaion07. View the logs.

4 years ago

This ticket was mentioned in Slack in #core by hilayt24. View the logs.

4 years ago

#6 @hilayt24
4 years ago

Keywords needs-testing added

As per the discussion in the *6.1* Bug-Scrub, the latest patch looks good. I am adding a needs-testing tag to keep things smooth. Thanks, @costdev for the suggestions.

Props: @costdev

Version 0, edited 4 years ago by hilayt24 (next)

This ticket was mentioned in Slack in #core by audrasjb. View the logs.

4 years ago

#8 @desrosj
4 years ago

Owner set to desrosj
Status new → assigned

#9 @desrosj
4 years ago

Resolution → fixed
Status assigned → closed

In 54241:

General: Throw a more descriptive error when templates are not found.

When wp.template() is called in JavaScript for a template element that does not exist, a very nondescript error message is currently returned (“Uncaught TypeError: Cannot read property replace of undefined”).

This updates adds a check for this scenario and a new “Template not found” error is now thrown instead.

Props joehoyle, noisysocks, hilayt24.
Fixes #36631.

Note: See TracTickets for help on using tickets.

Download in other formats:

Make WordPress Core

Context Navigation

#36631 closed enhancement (fixed)

wp.template() should throw an error if the template is not found

Description

Attachments (2)

Change History (11)

@joehoyle 10 years ago

This ticket was mentioned in ​Slack in #core by talldanwp. ​View the logs. 6 years ago

#2 @noisysocks 6 years ago

#3 @desrosj 4 years ago

@desrosj 4 years ago

This ticket was mentioned in ​Slack in #core by chaion07. ​View the logs. 4 years ago

This ticket was mentioned in ​Slack in #core by hilayt24. ​View the logs. 4 years ago

#6 @hilayt24 4 years ago

This ticket was mentioned in ​Slack in #core by audrasjb. ​View the logs. 4 years ago

#8 @desrosj 4 years ago

#9 @desrosj 4 years ago