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

Set the correct furniture id for box_large_reinforced #55256

Merged

Conversation

ccaviness
Copy link
Contributor

@ccaviness ccaviness commented Feb 10, 2022

Summary

Bugfixes "Set the correct furniture id for box_large_reinforced"

Purpose of change

It is incorrectly set to f_cardboard_box_reinforced, but it should be f_cardboard_box_large_reinforced

Fixes this debug error that appears when viewing the recipe:

 DEBUG    : invalid furniture id "f_cardboard_box_reinforced"

 FUNCTION : const T &generic_factory<furn_t>::obj(const string_id<T> &) const [T = furn_t]
 FILE     : src/generic_factory.h
 LINE     : 445
 VERSION  : 0.F-5987-g0d23870736

Describe the solution

Use correct furniture id.

Describe alternatives you've considered

Letting someone else fix it.

Testing

With this change, the error does not appear.

@github-actions github-actions bot added json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions labels Feb 10, 2022
It is inorrectly set to `f_cardboard_box_reinforced`, but it should be
`f_cardboard_box_large_reinforced`

```
 DEBUG    : invalid furniture id "f_cardboard_box_reinforced"

 FUNCTION : const T &generic_factory<furn_t>::obj(const string_id<T> &) const [T = furn_t]
 FILE     : src/generic_factory.h
 LINE     : 445
 VERSION  : 0.F-5987-g0d23870736
```
@ccaviness ccaviness force-pushed the f_cardboard_box_large_reinforced branch from c663a56 to edc3a22 Compare February 10, 2022 02:14
@github-actions github-actions bot added BasicBuildPassed This PR builds correctly, label assigned by github actions and removed BasicBuildPassed This PR builds correctly, label assigned by github actions labels Feb 10, 2022
@anoobindisguise
Copy link
Contributor

Agh, I'm a dummy. I even crafted one myself, I must have missed something somewhere, made a change and broke something. Thanks for beating me to it.

@Maleclypse Maleclypse added <Bugfix> This is a fix for a bug (or closes open issue) [JSON] Changes (can be) made in JSON labels Feb 10, 2022
@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Feb 10, 2022
@kevingranade kevingranade merged commit ee956f5 into CleverRaven:master Feb 12, 2022
@ccaviness ccaviness deleted the f_cardboard_box_large_reinforced branch February 12, 2022 01:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions <Bugfix> This is a fix for a bug (or closes open issue) [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants