Merge pull request #198 from Astrotomic/fix-private-to-protected

make all private methods protected

Author: Gummibeer

src/Translatable/Traits/Relationship.php

@@ -68,7 +68,7 @@ public function translation(): HasOne
             ->where($this->getLocaleKey(), $this->localeOrFallback());
     }
 
-    private function localeOrFallback()
+    protected function localeOrFallback()
     {
         return $this->useFallback() && ! $this->translations()->where($this->getLocaleKey(), $this->locale())->exists()
             ? $this->getFallbackLocale()

src/Translatable/Traits/Scopes.php

@@ -123,7 +123,7 @@ public function scopeWithTranslation(Builder $query)
         ]);
     }
 
-    private function getTranslationsTable(): string
+    protected function getTranslationsTable(): string
     {
         return app()->make($this->getTranslationModelName())->getTable();
     }

src/Translatable/Translatable.php

@@ -381,7 +381,7 @@ protected function saveTranslations(): bool
         return $saved;
     }
 
-    private function getAttributeAndLocale(string $key): array
+    protected function getAttributeAndLocale(string $key): array
     {
         if (Str::contains($key, ':')) {
             return explode(':', $key);
@@ -390,7 +390,7 @@ private function getAttributeAndLocale(string $key): array
         return [$key, $this->locale()];
     }
 
-    private function getAttributeOrFallback(?string $locale, string $attribute)
+    protected function getAttributeOrFallback(?string $locale, string $attribute)
     {
         $translation = $this->getTranslation($locale);
 
@@ -411,7 +411,7 @@ private function getAttributeOrFallback(?string $locale, string $attribute)
         return null;
     }
 
-    private function getFallbackLocale(?string $locale = null): ?string
+    protected function getFallbackLocale(?string $locale = null): ?string
     {
         if ($locale && $this->getLocalesHelper()->isLocaleCountryBased($locale)) {
             if ($fallback = $this->getLocalesHelper()->getLanguageFromCountryBasedLocale($locale)) {
@@ -422,7 +422,7 @@ private function getFallbackLocale(?string $locale = null): ?string
         return config('translatable.fallback_locale');
     }
 
-    private function getTranslationByLocaleKey(string $key): ?Model
+    protected function getTranslationByLocaleKey(string $key): ?Model
     {
         if (
             $this->relationLoaded('translation')
@@ -435,12 +435,12 @@ private function getTranslationByLocaleKey(string $key): ?Model
         return $this->translations->firstWhere($this->getLocaleKey(), $key);
     }
 
-    private function toArrayAlwaysLoadsTranslations(): bool
+    protected function toArrayAlwaysLoadsTranslations(): bool
     {
         return config('translatable.to_array_always_loads_translations', true);
     }
 
-    private function useFallback(): bool
+    protected function useFallback(): bool
     {
         if (isset($this->useTranslationFallback) && is_bool($this->useTranslationFallback)) {
             return $this->useTranslationFallback;
@@ -449,7 +449,7 @@ private function useFallback(): bool
         return (bool) config('translatable.use_fallback');
     }
 
-    private function usePropertyFallback(): bool
+    protected function usePropertyFallback(): bool
     {
         return $this->useFallback() && config('translatable.use_property_fallback', false);
     }