Voyager: Deleting BREAD displays incorrect button text

Created on 19 Mar 2018  路  6Comments  路  Source: the-control-group/voyager

  • Laravel Version: 5.6.11
  • Voyager Version: dev-1.x-dev
  • PHP Version: 7.2.3
  • Database Driver & Version: MySQL 5.7.21

Description:

The language file appears to not be referenced properly in some places, most notably during the popup for deleting a BREAD:

screen shot 2018-03-16 at 1 27 47 pm

The text for the Delete button is clearly incorrect. The button _does_ work, however. Yes, it's a minor issue, but might as well go ahead and document it here so it can be fixed someday.

Steps To Reproduce:

  1. With a fresh (or stale, for that matter) installation of Laravel and Voyager, create a database table and its associated Model.
  2. Create BREAD for that table.
  3. Attempt to delete said BREAD.
  4. Marvel at the incongruously-long DELETE button.
  5. ???
  6. Profit?

鈥ell, okay, perhaps you don't need those last two steps. ;~P

bug

All 6 comments

I recently splitted up the language files into groups, but the actually source looks like this
This should work properly. Check that you dont have any old files!

Okay, I updated Voyager with composer update so I should have the latest version now. During that process, it removed the old version of Voyager (and, presumably, all of its files, as far as I could tell). And, indeed, it does seem that the line of code you specified is identical to the one I have. However, the same thing appears, only with a slight change to the text on the button:

screen shot 2018-03-19 at 9 55 08 am

Okay, now thats a thing. Ref
It should be voyager::bread.delete_bread_conf
I will submit a fix to #2837

Fixed in #2837

Confirmed fixed. 馃帀

This issue has been automatically locked since there has not been any recent activity after it was closed. If you have further questions please ask in our Slack group.

Was this page helpful?
0 / 5 - 0 ratings