Add NO_DISPLAY vitamin flag, show non-nutritional vitamins as 'Other contents' in food information menu

data/json/vitamin.json

@@ -73,6 +73,7 @@
     "min": 0,
     "max": 40,
     "rate": "4 h",
+    "flags": [ "NO_DISPLAY" ],
     "disease_excess": [ [ 10, 19 ], [ 20, 29 ], [ 30, 40 ] ]
   }
 ]

doc/VITAMIN.md

@@ -1,4 +1,6 @@
-## vitamin
+# vitamin
+
+## definition
 
 ```JSON
 {
@@ -11,6 +13,7 @@
   "min": -12000,
   "max": 3600,
   "rate": "15 m",
+  "flags": [ "FOO" ],
   "disease": [ [ -4800, -5600 ], [ -5601, -6400 ], [ -6401, -12000 ] ],
   "disease_excess": [ [ 10, 19 ], [ 20, 29 ], [ 30, 40 ] ]
 },
@@ -55,6 +58,9 @@ The highest amount of this vitamin that the avatar can have.
 ### `rate`
 How long it takes to lose one unit of this vitamin.
 
+### `flags`
+An array of string flags, see the flags section below for valid ones
+
 ### `disease`
 What the thresholds of deficiency of this vitamin are.
 Each pair in the list determines the start and end points of that tier of deficiency.
@@ -64,3 +70,7 @@ Each tier of deficiency corresponds to the intensity level of the effect defined
 What the thresholds of excess of this vitamin are.
 Each pair in the list determines the start and end points of that tier of excess.
 Each tier of excess corresponds to the intensity level of the effect defined in `excess`.
+
+## flags
+
+- ```NO_DISPLAY``` - This vitamin will not be shown when examining a food

src/item.cpp

@@ -1359,16 +1359,35 @@ void item::food_info( const item *food_item, std::vector<iteminfo> &info,
                                           nutr.vitamins.end(),
     []( const std::pair<vitamin_id, int> &v ) {
         // only display vitamins that we actually require
-        return ( g->u.vitamin_rate( v.first ) > 0_turns && v.second != 0 ) ?
+        return ( g->u.vitamin_rate( v.first ) > 0_turns && v.second != 0 &&
+                 v.first->type() == vitamin_type::VITAMIN && !v.first->has_flag( "NO_DISPLAY" ) ) ?
                string_format( "%s (%i%%)", v.first.obj().name(),
                               static_cast<int>( v.second * g->u.vitamin_rate( v.first ) /
                                                 1_days * 100 ) ) :
                std::string();
     } );
+
+    const std::string effect_vits = enumerate_as_string(
+                                        nutr.vitamins.begin(),
+                                        nutr.vitamins.end(),
+    []( const std::pair<vitamin_id, int> &v ) {
+        // only display vitamins that we actually require
+        return ( g->u.vitamin_rate( v.first ) > 0_turns && v.second != 0 &&
+                 v.first->type() != vitamin_type::VITAMIN && !v.first->has_flag( "NO_DISPLAY" ) ) ?
+               string_format( "%s (%i%%)", v.first.obj().name(),
+                              static_cast<int>( v.second * g->u.vitamin_rate( v.first ) /
+                                                1_days * 100 ) ) :
+               std::string();
+    } );
+
     if( !required_vits.empty() && parts->test( iteminfo_parts::FOOD_VITAMINS ) ) {
         info.emplace_back( "FOOD", _( "Vitamins (RDA): " ), required_vits );
     }
 
+    if( !effect_vits.empty() && parts->test( iteminfo_parts::FOOD_VIT_EFFECTS ) ) {
+        info.emplace_back( "FOOD", _( "Other contents: " ), effect_vits );
+    }
+
     if( g->u.allergy_type( *food_item ) != morale_type( "morale_null" ) ) {
         info.emplace_back( "DESCRIPTION",
                            _( "* This food will cause an <bad>allergic reaction</bad>." ) );

src/iteminfo_query.h

@@ -34,6 +34,7 @@ enum class iteminfo_parts : size_t {
     FOOD_PORTIONS,
     FOOD_SMELL,
     FOOD_VITAMINS,
+    FOOD_VIT_EFFECTS,
     FOOD_CANNIBALISM,
     FOOD_TAINT,
     FOOD_POISON,

src/vitamin.cpp

@@ -78,6 +78,10 @@ void vitamin::load_vitamin( const JsonObject &jo )
         vit.disease_excess_.emplace_back( e.get_int( 0 ), e.get_int( 1 ) );
     }
 
+    for( std::string e : jo.get_array( "flags" ) ) {
+        vit.flags_.insert( e );
+    }
+
     if( vitamins_all.find( vit.id_ ) != vitamins_all.end() ) {
         jo.throw_error( "parsed vitamin overwrites existing definition", "id" );
     } else {

src/vitamin.h

@@ -5,6 +5,7 @@
 #include <map>
 #include <utility>
 #include <vector>
+#include <set>
 #include <string>
 
 #include "calendar.h"
@@ -48,6 +49,10 @@ class vitamin
             return name_.translated();
         }
 
+        bool has_flag( const std::string &flag ) const {
+            return flags_.count( flag ) > 0;
+        }
+
         /** Disease effect with increasing intensity proportional to vitamin deficiency */
         const efftype_id &deficiency() const {
             return deficiency_;
@@ -102,6 +107,7 @@ class vitamin
         time_duration rate_;
         std::vector<std::pair<int, int>> disease_;
         std::vector<std::pair<int, int>> disease_excess_;
+        std::set<std::string> flags_;
 };
 
 #endif