diff options
Diffstat (limited to 'crates/shirabe/src/package')
| -rw-r--r-- | crates/shirabe/src/package/dumper/array_dumper.rs | 4 | ||||
| -rw-r--r-- | crates/shirabe/src/package/package_interface.rs | 13 |
2 files changed, 9 insertions, 8 deletions
diff --git a/crates/shirabe/src/package/dumper/array_dumper.rs b/crates/shirabe/src/package/dumper/array_dumper.rs index 303a5871..336c96b0 100644 --- a/crates/shirabe/src/package/dumper/array_dumper.rs +++ b/crates/shirabe/src/package/dumper/array_dumper.rs @@ -92,8 +92,8 @@ impl ArrayDumper { data.insert("dist".to_string(), PhpMixed::Array(dist)); } - for type_name in SUPPORTED_LINK_TYPES.keys() { - let links = package.get_links_for_type(type_name); + for (type_name, opts) in SUPPORTED_LINK_TYPES.iter() { + let links = package.get_links_for_type(opts.method); if links.is_empty() { continue; } diff --git a/crates/shirabe/src/package/package_interface.rs b/crates/shirabe/src/package/package_interface.rs index d138fd83..c3cb1c56 100644 --- a/crates/shirabe/src/package/package_interface.rs +++ b/crates/shirabe/src/package/package_interface.rs @@ -191,17 +191,18 @@ pub trait PackageInterface: std::fmt::Display + std::fmt::Debug { /// @return array An array of package suggestions with descriptions fn get_suggests(&self) -> IndexMap<String, String>; - /// PHP helper that switches on the link kind (require/require-dev/conflict/etc.). + /// Stands in for PHP's `$package->{'get'.ucfirst($linkType)}()`, so `link_type` is one of the + /// `Link::TYPE_*` values. fn get_links_for_type( &self, link_type: &str, ) -> std::rc::Rc<IndexMap<String, crate::package::Link>> { match link_type { - "require" => self.get_requires(), - "require-dev" => self.get_dev_requires(), - "conflict" => self.get_conflicts(), - "provide" => self.get_provides(), - "replace" => self.get_replaces(), + crate::package::Link::TYPE_REQUIRE => self.get_requires(), + crate::package::Link::TYPE_DEV_REQUIRE => self.get_dev_requires(), + crate::package::Link::TYPE_CONFLICT => self.get_conflicts(), + crate::package::Link::TYPE_PROVIDE => self.get_provides(), + crate::package::Link::TYPE_REPLACE => self.get_replaces(), _ => std::rc::Rc::new(IndexMap::new()), } } |